Trojan Horse
lemmy.ml
external-link
An image tagged trojan horse

Please kill me

No, you still have a PR to review.

@_____@lemm.ee
link
fedilink
English
24
edit-2
25d

> someone nitpicks word you used in a variable declaration

> you change it

> someone more senior says the former made more sense

> this goes on for far longer than it should

> eventually you get a real review from someone in your team that identified something actually needs to change

> you change it and re request reviews

rinse and repeat

projectmoon
link
fedilink
825d

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…

People need to lameduck their code more

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

Fonzie!
link
fedilink
424d

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.

projectmoon
link
fedilink
424d

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

Fonzie!
link
fedilink
124d

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.

@mac@lemm.ee
link
fedilink
425d

Don’t forget get questioned by your manager/scrum lead as to why its taking so long to get out.

Well, I’ve had the PR ready for 3 days and the team asked me to make changes today

@Skullgrid@lemmy.world
link
fedilink
2
edit-2
24d

you forgot the part where you have to rebase your branch and that causes merge conflicts that were resolved later but somehow still persists.

Create a post

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

Rules:

  • Posts must be relevant to programming, programmers, or computer science.
  • No NSFW content.
  • Jokes must be in good taste. No hate speech, bigotry, etc.
  • 1 user online
  • 91 users / day
  • 116 users / week
  • 501 users / month
  • 2.48K users / 6 months
  • 1 subscriber
  • 1.6K Posts
  • 35.5K Comments
  • Modlog