In my previous post, Reference Guide: Conducting a Code Review, detailed steps were outlined for the code review process. Once the code review is completed it’s time to complete the process and make comments, approve the reviewed changes or request changes to be made before it can be approved. This is where any identified issues need to be noted.
To complete the pull request review:
Select green button
Review changes to finalize the pull request
- A pop-up box will appear with options to
The code review process is very important. If something needs to be changed or modified simply provide descriptive comments and select
Request changes. This will send a notification back to the contributor.
- Select the appropriate action:
Once submitted, you’ll be taken back to the
Pull request page.
Merging a pull request:
Provided everything on this page is
green, the changes are ready to be merged.
Merge pull request
- Once this has been selected the files will be merged and the icon under the pull request will switch from a
greenicon that states
purpleicon that states
Only a couple more steps!
Return to the terminal and complete the following:
git checkout master
These additional steps will pull changes from the master and update the local copy. It’s always best to make sure collaborators are working from current files.
For the completed Reference Guide Series:
- Part One: Reference Guide: Common Commands for Terminal.
- Part two: Create a GitHub Repository
- Part three: Committing Changes
- Part four: Committing Changes with Branches -Part five: Merge Conflicts
- Part six: Pull Requests
- Part seven: Conducting a Code Review -Part eight: Complete and Merge a Pull Request