DEV Community

Discussion on: React Hooks in 5 Minutes

Collapse
 
jh3y profile image
Jhey Tompkins

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

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