CTO at SourceLevel·

Do you review your Pull/Merge Request before assigning Reviewers?

If you work in a team opening a Pull Request (or Merge Request) looks appropriate. However, have you ever thought about opening a Pull/Merge Request when working by yourself? Here's a checklist of things you can review in your own:

  • Pick the correct target branch
  • Make Drafts explicit
  • Name things properly
  • Ask help for tools
  • Remove the noise
  • Fetch necessary data
  • Understand Mergeability
  • Pass the message
  • Add screenshots
  • Be found in the future
  • Comment inline in your changes

Read the blog post for more detailed explanation for each item :D

What else do you review before asking for code review?

READ LESS
10 items to check before assigning a Pull Request to someone - Plus a BONUS – SourceLevel (sourcelevel.io)
14 upvotes·649.8K views
Avatar of Weverton Timoteo

Weverton Timoteo

CTO at SourceLevel