this post was submitted on 13 Dec 2025
482 points (99.0% liked)
Linux
10808 readers
552 users here now
A community for everything relating to the GNU/Linux operating system (except the memes!)
Also, check out:
Original icon base courtesy of lewing@isc.tamu.edu and The GIMP
founded 2 years ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
I don't think you understand what doing code reviews is like.
So someone submits terrible code. You don't get to just say "this is bad code" and reject it wholesale, you have to explain in exhaustive detail what the problems are. Doing otherwise leads to really toxic environments. It's killed countless projects
That's why you write rules. You don't have to argue if they need tests or not, you tap the sign and reject it without actually reviewing it if it doesn't meet the requirement
Same thing here. You open up vibe coded nonsense so you tap the sign and reject it without bothering to review it. Do the same thing with "bad code" as a reason and it starts insane drama.
People are really sensitive about their code, and there's a whole methodology around how to do without ending up in a screaming match