logoalt Hacker News

t-writescode • yesterday at 4:39 AM • 1 reply • view on HN

Shouldn’t you be, in general, separating out new functionality from old fixes?

Shouldn’t those “fixing bugs we gained in the past” be their own MR that can be read, reasoned about and have evaluated test coverage?


Replies

01100011 • yesterday at 5:39 AM

Sure and sometimes that is what happens. Sometimes it's not. We should be doing a lot of things, but have to triage issues and act pragmatically.

In the case of what I'm currently working on, filing bugs for every issue I found, and then factoring out each fix, and then running each change through the 8 hour ci/cd system, and hoping an unrelated issue doesn't get misattributed to me... No, I'd rather just wrap it up into one coherent refactoring change and be done with it because when I'm done there are several more like it waiting for my attention.

➕ show 1 reply