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
-
If there is a written code standard you should follow it and if you do not it is a valid cause to reject the CR.
If its just some ones personal preference and not and agreed on standard, I agree with you that it should not be a valid reason.
It can feel ridiculous but if its written down there just might be a reason for it and if you disagree with the standard then bring it up as a separate topic.
Related Rants
code review
Code review time.
"How come this line has been removed? PEP 8 likes to have two lines between imports and the first bit of code"
What I replied: Thanks. I'll put it back.
What I wanted to reply: Go fuck yourself you anal moron, who the fuck gives a shit about bollocks like that. We got fucking proper work to do, so get the fuck over yourself, let the fucking PEP shit lie, and make some fucking USEFUL comments.
devrant
code review
pep8