this post was submitted on 28 Nov 2024
518 points (98.5% liked)

Programmer Humor

32725 readers
125 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

founded 5 years ago
MODERATORS
518
Lean vs. fast (lemmy.ml)
submitted 3 weeks ago* (last edited 3 weeks ago) by [email protected] to c/[email protected]
 
you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 8 points 3 weeks ago (2 children)

Had a team lead that kept requesting nitpicky changes, going in a FULL CIRCLE about what we should change or not, to the point that changes would take weeks to get merged. Then he had the gall to say that changes were taking too long to be merged and that we couldn't just leave code lying around in PRs.

Jesus fucking Christ.

There's a reason that team imploded....

[–] [email protected] 4 points 3 weeks ago (1 children)

Had a colleague who would comment things like “add a newline here” as well as things that were fully his own preference.

That was the only time I closed comments without replying to them or fixing them, without feeling bad.

[–] [email protected] 4 points 3 weeks ago (1 children)

For stuff like that, it's best to have an auto formatter like checkstyle or something.

[–] [email protected] 1 points 3 weeks ago

My point exactly! But naw, several others on the team insisted this guy policing others manually is better than putting a linter in the pipeline.

I don't work there anymore, this is one of the (minor) reasons.

[–] [email protected] 1 points 3 weeks ago (1 children)

People need to lameduck their code more

[–] [email protected] 2 points 3 weeks ago

People need to reply to those comments with "out of scope" and a link to a new issue that will get buried in the backlog more often