this is terrible advise - you should be using unknown
. using any
you’re basically disabling TS and will be under the false assumption that your code is ok while it’s most likely missing a lot of runtime checks
using any
is actually much worse than using TS, because you’re basically telling the compiler “don’t help me here”… at least with JS the IDE is gonna help you… :/
this, and also nothing is 100% new - knowledge in similar areas will always help
haha same, I wish we could give names to backpacks and pouches!
who should we be worried about?
yeah, fair point. it really only works with standard boilerplate code which is simple enough to not have any issue I guess… in my case working with a NX monorepo, that would be any code created using the generators
That’s why PR should be small. It’s much better to have multiple PRs than a single big one.
Totally fair to have gigantic PR full of boilerplate code, but generally you can split the boilerplate and your feature in 2 PRs, where only the feature will get a proper review.
All of this obviously depends on the criticality of the system :p
there are shows where it works (Frasier) and shows where it’s horrible (Frasier 2023)