Comments by "Brent Jacobs" (@br3nto) on "STOP Nit Picking In Code Reviews" video.

  1.  @ThePrimeTimeagen  lol yeah… of course… but also some logic to this… if a dev is not caring about the small things, they probably aren’t caring about the big/important things either. Or if they do, they probably aren’t getting the big things right either. My guess is there’s a correlation there.
    4
  2. Nah, nitpicking is good. If you don’t fix it now, it’ll sit around annoying everyone forever… inconsistent casing, spacing, indenting, naming, use of concatenation instead of interpolation, use of concatenation instead of multi-line stings, no new line at the end of files, EOL whitespace, new lines not added between methods and between logically grouped statements and expressions… all of it should be closely guarded and protected and fixed.
    3
  3. 4:46 is the reverse true? Is it a nit to get people to remove leading _ from private fields when they add them?
    1