967
Review Please (programming.dev)
submitted 5 months ago by [email protected] to c/[email protected]
you are viewing a single comment's thread
view the rest of the comments
[-] [email protected] 26 points 5 months ago

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.

[-] [email protected] 2 points 5 months ago

Very good point. We often do one PR per story so people tend to think that's a limit.

this post was submitted on 29 Mar 2024
967 points (98.3% liked)

Programmer Humor

19207 readers
1180 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS