Stupid dev creates mega-PRs that do a ton of things that are not part of the ticket and thus aren’t tested by QA.
When it then inevitably fails during release testing or in production, better devs need to spend quite a bit of time to figure out what the hell the stupid dev messed up and the fix is then often just a few lines.
We have two teams working on the same codebase. The teams are very far apart in terms of organization. We are in different departments.
That dude is in the other team. Each team does their own code review and is supposed to ask for cross-team review for stuff that affects the other team.
The guy built himself a reputation as an infallible master programmer in his team by bullying everyone else during PRs. You know, nitpicking tiny non-mistakes like variable names, blocking PRs for weeks by always finding some unnecessary detail, then waiting for a few days after that detail was adjusted to find another one and so on. So his team thinks he’s the greatest. Or maybe they are afraid to be his next victim or something. Whatever the reason, they just greenlight everything he does. And he never asks for a cross-team review.
His boss also completely covers for him, so so far our escalations just went nowhere.
We have that situation at work.
Stupid dev creates mega-PRs that do a ton of things that are not part of the ticket and thus aren’t tested by QA.
When it then inevitably fails during release testing or in production, better devs need to spend quite a bit of time to figure out what the hell the stupid dev messed up and the fix is then often just a few lines.
What are the reviewers of these PRs doing? Out-of-scope PRs can be rejected and closed, no?
It’s a really weird team mechanic.
We have two teams working on the same codebase. The teams are very far apart in terms of organization. We are in different departments.
That dude is in the other team. Each team does their own code review and is supposed to ask for cross-team review for stuff that affects the other team.
The guy built himself a reputation as an infallible master programmer in his team by bullying everyone else during PRs. You know, nitpicking tiny non-mistakes like variable names, blocking PRs for weeks by always finding some unnecessary detail, then waiting for a few days after that detail was adjusted to find another one and so on. So his team thinks he’s the greatest. Or maybe they are afraid to be his next victim or something. Whatever the reason, they just greenlight everything he does. And he never asks for a cross-team review.
His boss also completely covers for him, so so far our escalations just went nowhere.
deleted by creator
Sorry, what’s that? Google was of no help.
Well, he’s known to develop the same way and a whole team has to clean up his mess. Just what it reminded me of, maybe it’s inapropriate.