The Naming Police
ELI5 - The Vibe Check
The function works. The tests pass. The logic is sound. But the review has four comments, all about whether it should be 'getUserData' or 'fetchUserData.' The Naming Police is the reviewer whose entire contribution to any pull request is opinions about naming, delivered with total confidence and zero comments on whether the code actually does the right thing.
Real Talk
The Naming Police is a colloquial term for a code reviewer who focuses disproportionately on naming conventions and style at the expense of reviewing logic, architecture, or correctness. Naming consistency has genuine value for readability, but when it becomes the sole focus of review, it can crowd out feedback that would catch actual bugs.
When You'll Hear This
"The Naming Police struck again, four comments on variable names and nothing about the missing null check." / "I appreciate good naming feedback, I just want the Naming Police to also read the function body sometimes."
Related Terms
Bikeshedding
The nuclear power plant committee approved the reactor design in 2 minutes — no one understood it well enough to debate. The bike shed for employees?
Code Review
A code review is when another developer reads your code before it gets merged, looking for bugs, bad practices, or anything confusing.
Code Smell
Code that works but feels... wrong. Like when food smells slightly off — it might be fine, but something isn't right.
Nit
A nit (short for nitpick) is a tiny, non-blocking code review comment about something that doesn't really matter but someone felt strongly enough to mentio...
Rubber Stamping
Approving a pull request without actually reading the code. Just clicking 'LGTM' and moving on. We've all done it. We've all been burned by it.