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 👍
I think using an empty array looks cleaner and more readable.
Yeah, definitely 👍 But it seems to be discouraged if the effect in question makes use of props + state in the component's scope.
We're a place where coders share, stay up-to-date and grow their careers.
We strive for transparency and don't collect excess data.