Leebra
login register
That post seems to be missing.
load all comments
tdawg 10 points 3 years ago

Honestly refactoring everything isn't an "improvement." More often than not it's a sign of that developer not understanding what the code does. Which is fine if they need to do that in a new code base or are otherwise rather green. Writing code is easier than reading it, after all. However they really should stash those changes or reopen a new branch after that exploratory phase.

IMO (in general) a good bug fix makes the smallest possible change to implement said fix. Otherwise the focus is about documention of the bug if needed and some minor refactoring to improve readability, consistency, or efficency

save

path: 0 1584726, hotness: undefined, score: 10, children: 3
ngcbassman 13 points 3 years ago

I like the approach of when finding a bug, write a test to reproduce it, fix the bug so the test passes, next do whatever refactoring you want because you have a test passing :)

save

path: 0 1584726 1586414, hotness: undefined, score: 13, children: 1
tdawg 2 points 3 years ago

This sounds like a great technique. I'll have to give it a go on Monday haha

save

path: 0 1584726 1586414 1601643, hotness: undefined, score: 2, children: 0

thanks for using Leebra!

go to feed...