|
|
|
|
|
|
by jpgvm
1644 days ago
|
|
|
I always prefix such comments with "nit" so the owner of the PR knows I'm not saying they should/must make this change to get a +1 from me but that I think it should be considered. That said we already use auto linters and formatters. I am more mindful of my code review comments now that I'm more senior. When it's easy for people to take my words as gospel I much prefer to foster an environment where that isn't the case. |
|
|