Joe Lencioni 0f84b0f02b Fix ExhaustiveDeps ESLint rule throwing with optional chaining (#19260) ...

Certain code patterns using optional chaining syntax causes
eslint-plugin-react-hooks to throw an error.

We can avoid the throw by adding some guards. I didn't read through the
code to understand how it works, I just added a guard to every place
where it threw, so maybe there is a better fix closer to the root cause
than what I have here.

In my test case, I noticed that the optional chaining that was used in
the code was not included in the suggestions description or output,
but it seems like it should be. This might make a nice future
improvement on top of this fix, so I left a TODO comment to that effect.

Fixes #19243

2020-07-06 15:52:14 -04:00

..

ESLintRuleExhaustiveDeps-test.js

Fix ExhaustiveDeps ESLint rule throwing with optional chaining (#19260)

2020-07-06 15:52:14 -04:00

ESLintRulesOfHooks-test.js

Added clear message for functional component starting with lowercase (#18881)

2020-05-10 01:52:11 +01:00