New option: ignoredFunctions

[karfau]

Adds a new option to configure function names that should not be treated as hooks. Helpful with functions that have been created before hooks existed, so al the benefits of the rule apply ignoring those until they have been renamed.

Still untested, wanted to first know if this goes into the right direction before adding extensive testing.

I would like to get first feedback about this idea and the way I attempted to implement it:

• Please let me know if this is something you would like to add.
• Any feedback/ concerns about the place where I'm applying the new option?
• (Does it even make sense to allow ignoring things from the React namespace?)

[karfau]

I agree and in the meantime I also tried to rename the offending functions in our code base, so we solved the issue by a refactoring. (Which might not have happened if the option would have been available.) I think this is also the way for people to go if they want to have this tslint rule.

Also since tslint is going to ~dy~be deprecated I have a hard time motivating myself to invest more time into anything that is not related to moving away from it...

Thx for the feedback, hope to collaborate some time in the future :smile:

[Gelio]

Sure, no problem, thanks for contributing and an intellectual exchange of opinions 😄