Skip to content

Steps to a successful PR

Weiyuan Wu edited this page Apr 27, 2020 · 10 revisions

When submitting a PR, please follow these steps:

  1. You finish the code and ensure it has a good quality.
  2. R(ebase)-S(quash)-R(eword): You rebase the commits onto the newest develop branch, squash the unnecessary commits and merges and reword the commit message. You make sure the commit messages conform to the coding rules and are informative as release notes (we generate release notes directly from commit messages).
  3. You create a PR and fill in the PR template carefully. If any UI changes are involved, a screenshot should also be included. If an issue is closed by this PR, add closes #<issue-id> in your PR text.
  4. You wait until CI passes. If CI failed, change the code until CI passes and ensure RSR.
  5. You request a code review from our team members. You get several comments later and change the code to resolve the comments. You also click the resolve button on the comments.
  6. You ensure RSR, and again you request the code review.
  7. The reviewer approves your PR and he clicks the merge button.
  8. Congratulations! You successfully contribute to Dataprep and we really appreciate that!
Clone this wiki locally