undefined | Better HN

0 pointsbb881y ago0 comments

Worse is when a protocol or shared state condition is modified.

E.g. suddenly some fresh out of college know-it-all sent crap into your function that you weren't expecting. Then he went to management to blame you for writing such shitty code.

Thing is you wrote unit tests around that code and the shitty know-it-all deleted them rather than changing them when he modified the code

This is why management needs to understand code.

0 comments

margalabargala1y ago

Recently on HN there was a thread debating the utility of having required code reviews for PRs.

I'm firmly on team "require a coworker to say okay before merging", and this is exactly why.

josephg1y ago

What? Is that a real example? Are you seriously working with people who delete your tests, misuse your code then complain about you to management?

Is your workplace filled with high school students? I’ve never seen anything so petty and immature in my professional career. I hope management told them to grow up.

bb88OP1y ago

> Is your workplace filled with high school students? I’ve never seen anything so petty and immature in my professional career.

I think the description I remember on glass door was that it was "high school all over again".

j / k navigate · click thread line to collapse