Programmer Humor

20039 readers

858 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 2 years ago

MODERATORS

949

Review Please (programming.dev)

submitted 10 months ago by [email protected] to c/[email protected]

97 comments fedilink hide all child comments

you are viewing a single comment's thread
view the rest of the comments

[–] [email protected] 25 points 10 months ago (1 children)

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.

[–] BrianTheeBiscuiteer 2 points 10 months ago

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