Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
Just add a formatter to the pipeline - or to your git hooks.
More than once I told people that thinking and trying to invent clever rules for naming, indentation or the general "but I like it more this way" type of reasoning takes away more time than it actually saves while reading the code.
Especially if people make it a habit to nitpick in PR.
Thus the usage of GIT hooks. Format it *before* it lands in an PR.
Anyone that then starts nitpicking gets a link to the GIT hook documentation page and a "code is formatted according to standards and documentation" comment.
Plus if it accumulates, they land on my naughty list, meaning I have a very close eye on them, because these kinds of delays make it hard and frustrating for everyone. -
var x = some data
var y = more data
var bug = I don't know
---
Reviewer: There is a bug in your code, fix it!
Imagine that happens for real lol -
Fuck your indentation opinions. Fuck your tab/spacing opinions.
Does it work you douchebag?
Related Rants
-
DasKoder15I was told during my initial interview that the book "Clean Code" is their Bible here. And it's true. It's ly...
-
NuflowX34Seriously, why? WHHHYYYY? US-date-format sucks, every FUCKING TIME! The only time I really notice is when th...
-
Chrzanek7When you're preparing for a hard disk format:
To my vigilant code review overlords, yes, I appreciate the scrutiny, but must we nitpick every variable name? I've been told my camelCase is more like llamaCase.
Let's focus on real bugs, not whether my indentation is an affront to nature.
Remember, the perfect code doesn't exist – unless you're coding on a cloud XD
rant
variable
indent
clean code
formatting
indentation
clean
format
camelcase