News for hackers. Stuff that matters.
Re:The reality of long-term software maintenance from the ma... by nigels-com on Wednesday February 19, 2025 @08:14AM (#43101777)
I tend to be picky about code reviews. I'm aware that it causes friction and can come across as unreasonable. It works, we're done, right? Well it's likely to be my problem to maintain this code at some point in the future. Overly complicated? Unstated assumptions? Lack of test coverage? Fits well with nearby things? Solving a problem is fun but software engineering is more than just the fun parts.
Re:The reality of long-term software maintenance from the ma... by senkora on Wednesday February 19, 2025 @11:51AM (#43104274)
+1. Back when I was interning at various places, the conclusion that I came to was that it almost never made sense to push back on changes, because the reality was that I would be gone in 3 months and the reviewer would be stuck with that code for years. Even if I disagreed on something, the right thing to do was to clearly state my reasoning and then change it to whatever they wanted.
Re:The reality of long-term software maintenance from the ma... by jamesfinlayson on Wednesday February 19, 2025 @10:47PM (#43110872)
Ugh, had this argument recently, to my detriment.

Tests are great in theory said my manager, but if the new doesn't want to write them, why are you making such a fuss. Fast forward three weeks and new guy is gone.