DEV Community

loading...

Discussion on: React Hooks in 5 Minutes

Collapse
jh3y profile image
Jhey Tompkins Author

Good spot Pablo! In that case, it's not necessary and they could be omitted 👍
The docs almost discourage using an empty Array for the optional dependencies list and eslint-plugin-react-hooks's exhaustive-deps rule will in most cases pipe up when using useEffect. That's because in most cases, our effect uses props + state from the component scope. There's a good piece on using the empty Array and why better solutions are normally the way to go here 👍

Collapse
ptejada profile image
Pablo Tejada

I think using an empty array looks cleaner and more readable.

Thread Thread
jh3y profile image
Jhey Tompkins Author

Yeah, definitely 👍 But it seems to be discouraged if the effect in question makes use of props + state in the component's scope.