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
-
@linuxxx just one Sprint, then I'll work on the new system again
-
I once had a job where my code reviews had to go thru a certain "engineer". His only complaint was extra white space. I even threw in obvious code smells and errors to see what he could catch.
I always suspected he was a poser and couldn't write code to save his ass...
White space is a great separator -
@CoffeeNcode
Cmon. Its not the first time they bullshit you. Get outta there. -
@gathurian the new system: yes thru have pipelines. The old one: no. We're busy getting rid of it gradually, but it's a huge codebase
-
@scor believe me, this job is incredibly sane compared to my last one
-
@Swish "Nota bene", latin for "please note"
Related Rants
Sometimes happens with my own code !
me: *submits a PR on our old system*
My teamlead: please mind the code style, there was no need to add the extra whitespace line under that loop.
Me: 🤔 there are style conventions in this project??
-----
NB: this is really old code with tons of long lines, weird spacing and whitespace in random places and functions that span hundreds of lines.
I seriously did not notice a consistent style here.
I wanna go back to working on the new system ☹️
rant
spaghetti code
code style