How to do pull request reviews

• Code repository hosting service

• Branching

• Creating the PR

• Reviewing the PR

• Merging the PR

Github versus Bitbucket

Two of the most popular services for hosting teams’ code repositories are 

 and 

. Check the 

 and 

 for Github and the 

 and 

 for Bitbucket to see which one is more suitable for your team. If your team is already immersed in the 

 ecosystem (e.g. using 

 for their project management software and 

 for their documentation), it makes sense to also use Bitbucket because it integrates well with Jira.

What we use at Mage

We use Github to host our repositories. It’s hands down the most popular hosting service used by developers and easy to onboard new team members since they are probably already familiar with it. The Github Team pricing plan is sufficient for a team of less than 10 developers.

• Mage’s branch-naming convention (without brackets): 

  [initials_or_team_name]--[your_branch_title_snake_case]

  .

• Prefix commit messages with your initials in brackets: 

  [initials]

  .

1
2
3
4
$ git checkout -b [initials_or_team_name]--[your_branch_title_snake_case]
$ git checkout -b jk--new_sign_up_flow< make some code changes >
$ git add .
$ git commit -m '[initials] <title of your changes>'

These are the requirements for team members creating new PRs:

• Add at least one reviewer (adding multiple reviewers may not be allowed in Github’s 

  ).

• Assign yourself and anyone else who worked on the PR as assignees.

• Include a brief summary of what was done. This could be bullet points.

• Include screenshots of the new features if applicable (e.g. adding a feature to the UI on the frontend).

• Include sample API request payloads if applicable. It will be easier for the reviewer to check that the payload is formed correctly.

• Mention any tests that were added or performed.

• Prefix the title of the PR with your initials (e.g. 

  [jk]

  ) so that the initials appear in the commit message in the 

  main

   branch’s commit history after the current branch is merged.

The reviewer goes through the changes and makes any comments as needed. There may also be some non-blocking comments (e.g. minor comments that don’t require any fixes) that the assignee should be aware of but don’t require further action. Then the assignee addresses all of the comments and informs the reviewer that the PR is ready for a follow-up review. Once the PR is ready to be merged to the 

main

 branch, the reviewer approves the PR.

An approval from a reviewer is required before being able to merge the PR. Github can be configured so that an assignee cannot actually merge the PR without that approval (the merge button will be disabled).

Before merging the PR to the 

main

 branch, there should be no conflicts with it, and all tests as part of the 

 should be passing on the current branch. We’ve written a separate 

 on how we set up our CI/CD pipeline.

There are 

 to merge a branch in Github:

1. Merge commit — All commits from the current branch are added to the base branch, and the PR is merged using the 

   --no-ff

    

   .

2. Squash and merge — The PR’s commits are squashed into a single commit, and the PR is merged into the base branch using the 

   .

3. Rebase and merge — All commits from the current branch are added onto the base branch individually, and the PR is merged using the 

   .

We use the “squash and merge” method for merging the current branch into the 

main

 branch. This can be configured to be required in the Github repository 

. We prefer to squash commits because it keeps the 

main

 branch’s commit history cleaner and easier to read.

After the current branch is merged into 

main

, delete the current branch. This can be done automatically in 

.