Skip to content

Steps to a successful PR

Weiyuan Wu edited this page May 14, 2020 · 10 revisions

When submitting a PR, please follow these steps:

  1. The code meets "small CL" requirement and 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 merge commits and reword the commit messages. 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). Attention: Do not have fix commits that fix bugs from the same PR!

  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!

  9. [Internal Only] For internal contributors (those who create branches into dataprep repo), please keep your branch name following the git workflow convention. E.g. a feature branch should named as feat/<feature_name? and a fix branch should be fix/<fix_name>.

Clone this wiki locally