seen it? seen it recently?
then your QC is either suffocatingly strict, or completely dead and probably doesn't even exist, lol.
no exceptions, sozzas.
here's the thing, you can't have overly strict requirements around your PR process. when you do that you're inviting PR trading, people skimming over thousand line changes, bottlenecking the team, and worst of all hurt feelings
but come on, you also cant have what basically amounts to commiting to master! there needs to be some kind of approval process in this place, right?
there's a solution, my friend.
it's to be a professional developer. you want change requests, they make you think, and reflect upon and evaluate your positions and opinions as a developer to better convey them, and understand them more deeply yourself.
they make you BETTER. 💪😎👍
if my code sucks, PLEASE request changes. if my code has tiny little issues, PLEASE nitpick it. if my approach is inconsistent with our standards, or best practices, PLEASE reject it.
being nitpicked or having changes requested in front of your peers really isn't embarrassing. it's not calling into question your abilities as a developer. it's someone trying to help you improve, and keep the codebase in alignment.
alright, cool.
if you think they're wrong in their request for changes then this is your chance to slap back with a hot comment and get that worm on the other foot, yo.
once you've proven them wrong (nicely), you can proceed to gossip about them in a Slack DM.