I've almost entirely stopped "nitpicking" in code review, but one nit-picky thing I've held onto is avoiding meaningless implicit return values from arrow functions that perform side effects. React 19 release notes include a great example of why I think this very common pattern should be avoided.
Comments