JPDev@programming.dev to Programmer Humor@programming.dev · 8 months agoReview Pleaseprogramming.devimagemessage-square112fedilinkarrow-up1991arrow-down117
arrow-up1974arrow-down1imageReview Pleaseprogramming.devJPDev@programming.dev to Programmer Humor@programming.dev · 8 months agomessage-square112fedilink
minus-squareJesus_666@feddit.delinkfedilinkarrow-up26·8 months agoOr, if the team does allow refactoring as part of an unrelated PR, have clean commits that allow me to review what you did in logical steps. If that’s not how you worked on the change than you either rewrite the history to make it look like you did or you’ll have to start over.
minus-squareBrianTheeBiscuiteer@lemmy.worldlinkfedilinkarrow-up2·8 months agoVery good point. We often do one PR per story so people tend to think that’s a limit.
Or, if the team does allow refactoring as part of an unrelated PR, have clean commits that allow me to review what you did in logical steps.
If that’s not how you worked on the change than you either rewrite the history to make it look like you did or you’ll have to start over.
Very good point. We often do one PR per story so people tend to think that’s a limit.