Skip to content

Reviewing pull requests

Giving feedback on pull requests, i.e. reviewing them, is a very important contribution too. While it's mostly core contributors doing reviews, don't shy away from a review if you aren't one! Anybody can review pull requests, only core contributors can merge them.

Tip

Reviews also show up as green blobs under your GitHub profile, did you know that? :)

Guidelines for reviewing code

For general guidelines on reviewing code, see the Dojo Library. Stack Overflow also has some good principles.

Accessing the code of a pull request locally

If the PR is coming from a fork, working with its code locally won't be as trivial as doing a git checkout on the branch of a core contributor. See GitHub's docs on this.

Tips on managing pull requests

Note

Adding labels and merging pull requests is only available for core contributors. Otherwise, anybody can review pull requests.

  • Here's a list of open PRs reviewed by you. Maybe check them because perhaps it's time to ping the authors since maybe they forgot to follow up. Or maybe you forgot to merge a PR?
  • Add the "don't merge" label on PRs that are ready for review, approved by you and/or others, but you asked more people for feedback before it can be merged.
  • Add the "needs triage" label on PRs that you think should be checked by others during the weekly triage meeting too. Explain in a comment why you deem this is necessary.
  • Be sure to merge PRs that don't need a second opinion. This is especially important for external contributors who can't merge their PRs. Keeping PRs open will make them collect merge conflicts and make the contributor demotivated.
  • Be especially patient and encouraging with first-time contributors. This is indicated by such a flag on the PR.
  • You can add the Feedback section to your review comment (i.e. the one that you send your per-line comments with) to remind the author about the practices.

Backporting Pull Requests to Another Branch

Occasionally, you may find it necessary to cherry-pick a pull request into a release branch after it has been merged into main. To facilitate this process, we've established a dedicated workflow called backport.

To initiate the backporting process, simply leave a comment on the pull request you wish to backport. The comment should contain the following command: /backport to release/1.8. This command serves as a trigger for the backporting workflow, seamlessly bringing the desired pull request into the specified branch.