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
-
Furthermore, in the code i reviewed was something like a different update endpoint for each combination of properties of an object.. which in the end where like 15 endpoints.
Who has the time to write so much garbage? There wasn't any need to even have all that combinations, let aside writing them right in a single endpoint. -
@OldManOfTheC this is true lol. My manager insists that we need to move faster, bad codes will be tolerated with a todo comment. Of course no one fixes them
-
@KennyTheBard did he consult you or someone with more experience about the approach before? Telling someone more experienced than you about what you plan to do in a PR often saves lots of CR time
-
@Xamenyap nope. we had the main outline of the project and what needs to be done, but the implementation itself is wild west, each for their own.
But the thing is that we both had the same experience on paper, including the studies (down to the college). So we should somehow be on the same page on the technical knowledge
Related Rants
-
sariel20Simple 1 day task. This idiot takes two weeks and after 7 days of hounding finally opens a pull request. I go...
-
PaperTrail19My code review nightmare part 3 Performed a review on/against a workplace 'nemesis'. I didn't follow the depa...
-
Root7The nightmare continues. Currently dealing with a code review from a “principal” dev (one step above seni...
Wasting about an hour trying to leave detailed comments and suggestions for a fellow student on an university project's code.
Said person just respond on the team group with "If anyone does not like something, he/she can go ahead to change it, so we would move on faster"
THEN WHY THE FUCK ARE WE STILL DOING CODE REVIEWS???
rant
wk279