[siquick]

I had a Lead like this. It go to the point where I would spend a significant portion of my working time in existential dread about submitting a PR because I knew they would just rip it to pieces. The worst part was small styling issues that would have been picked up by any linter - but the lead refused any requests for the team to use linting because “we have never used it and everything works fine”

I lasted 4 months before resigning but I took longer to get my confidence back.

[pydry]

Your lead was a junior.

One of the reasons I always fought to have black from almost the day it existed was because (I thought) it would put an end to this type of petty human linting.

[peakaboo]

I have to disagree with the view that not using black means the lead is not senior.

Being senior means you understand the tradeoffs and you make your own decision.

If you ask a senior why he choose not to use kubernetes, you will get a bunch of real reasons. If you ask a junior the same, he won't have a good answer because he lacks experience with the pros and cons.

[nucleogenesis]

Right and “we don’t need to use black because we haven’t yet and things work fine” is a bad response that dismisses the real needs and concerns of others on the team when they’re saying “things aren’t fine we want computers to handle the nitpicks from here”. I wouldn’t say it indicates junior tech skills but it’s a sign of junior management skills.

[Aqueous]

I would say it indicates junior tech skills as well. If the value of the linter isn’t self evident to him he is a junior.

[pydry]

>I have to disagree with the view that not using black means the lead is not senior.

I disagree with this straw man too.

It's the code review focus on small almost irrelevant stylistic details that signals a lack of experience.

[bracketslash]

what is “black” in this context?

[xmcqdpt2]

The python formatter, I guess.

https://github.com/psf/black

[amhoab]

https://github.com/psf/black

[bewestphal]

Agreed styling code comments are asinine. Teams need a common linter to leave it to the CI to determine the style, whatever it may be. It’s a waste of everyone’s time.

[jpgvm]

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.

[tossaway9000]

I don't know why someone would want to waste time arguing about code style.

Standardizing on things like `cargo fmt`, `go fmt`, and `terraform fmt` remove a ton of nitpicking out the gate. The javascript world can't seem to make up their mind though (jslint is rarely used these days, jshint died, I think eslint is the thing now?)

[Hasnep]

Eslint is a linter, the most common formatter for JS that I've seen is Prettier.