Power dynamics. Usually the person making the giant PRs is the one with all the sway. An earlier-career engineer is unlikely to push back against that level of influence.
PRs are all about power dynamics and (un)spoken deals…
If you rubberstamp some people‘s PRs all the time, you can then get them to greenlight your unpleasant PRs via pm instantly.
The other way round, retaliation: I once added some serious review notes to the PR of a very senior engineer because it was a dangerous topic. He would then spend the next months nitpicking every single PR I created. Had to post my PR in slack whenever he was not online to get them merged. After that I never seriously reviewed his PRs again. Too much of a headache.
It can be a company wide policy rather than trying to target a single individual even if the outcome is that they are targeted. This is something that should be addressed to them through a manager etc or if not, it's time to leave while they ruin the product over time.
Perfect example of a non-sequitur. Irrespective of whether or not the statement is true, it has no bearing on the veracity of the original claim: that in the current market, the majority of workers simply do not have this leverage.
Same can be said back to you. Obviously mine and 50+ acquaintances experience is not the entire world but geographical clusters and/or work-area clusters do apply.
Can you pick up the phone and be in the next job the next week?
Well, I am upset. But -- bubbles. My health has been in decline for years and I am very slowly making some strides. Missed on a million networking opportunities by being unable to go to conferences and... yeah, I'll not turn this into a sob story.
LLMs really accentuated the importance of networking, did they not?
What's the alternative? You push back or you don't, leaving you likelier to leave in the future. For non-junior devs, the market is still humming along.
I don't see how the market is humming along for anyone.
Then again, I have zero network. Maybe you can just call someone on the phone and jump ship next week? I can't. Many other people cannot as well.
My idea right now is to find ways to do things mostly my way and introduce a near-perfect meritocracy in my team. No seniors or juniors; I am technically "the most senior" but we all have differing and unique experiences. I share my experiences and when I feel stronger about something I make it clear why but I don't go sad in the corner if the other engineers overrule me.
Regardless of how the market is, I like getting along with people. Of course sometimes (actually: often) it's not possible in which case either a team restructuring should be done, or one should indeed leave (which is the nuclear option; not just "oh well, things did not work out").
> I don't see how the market is humming along for anyone. Then again, I have zero network
I mean, yeah, that's the issue. Even without a network my LinkedIn is full of recruiter spam because my profile is optimized, which is a skill anyone can learn and do, same concept as SEO.
> I like getting along with people.
Sure, who doesn't? The issue is when one becomes a doormat, just as in other social situations; in this case, it'd be being nice to others when in reality you'd need to be firm but fair that their writing huge PRs is negatively affecting everyone else. It's the paradox of tolerance applied to the engineering world.
Yeah, you are sadly right. Took me a long time to stop being a doormat. Not sure I am nailing it even now to be honest, but I am doing _much_ better compared to two years ago.
And honestly, even if you do push back, you probably can't succeed. I used to work with a guy that made enormous, 10k line PRs to our Jenkins code, and would give only 3-4 days for people to review it. We tried to push back on it, but he was the golden boy of one of the people in charge of the project. Even the (inevitable) breakage of software builds when he merged his changes didn't cause any consequences for him. Unfortunately, sometimes with office politics there's absolutely nothing you can do.
If you rubberstamp some people‘s PRs all the time, you can then get them to greenlight your unpleasant PRs via pm instantly.
The other way round, retaliation: I once added some serious review notes to the PR of a very senior engineer because it was a dangerous topic. He would then spend the next months nitpicking every single PR I created. Had to post my PR in slack whenever he was not online to get them merged. After that I never seriously reviewed his PRs again. Too much of a headache.