JPDev@programming.dev to Programmer Humor@programming.dev · 8 months agoReview Pleaseprogramming.devimagemessage-square102fedilinkarrow-up1968arrow-down117
arrow-up1951arrow-down1imageReview Pleaseprogramming.devJPDev@programming.dev to Programmer Humor@programming.dev · 8 months agomessage-square102fedilink
minus-squareswordsmanluke@programming.devlinkfedilinkarrow-up14·8 months agoNet removal of 1500 LoC… I’m gonna make you break this up into multiple PRs before reviewing, but honestly, if your refactoring reduced the surface area by 20% I’m a happy man.
minus-squareGraz@feddit.delinkfedilinkarrow-up1·8 months agoit’s just removed unit tests that didn’t work any more…
Net removal of 1500 LoC…
I’m gonna make you break this up into multiple PRs before reviewing, but honestly, if your refactoring reduced the surface area by 20% I’m a happy man.
it’s just removed unit tests that didn’t work any more…
😱