Every MarkUs developer has to perform a Code Review before their code can be merged (even for the smallest commit). This is a short guide as to how to use the Github Code Review system.
Once you have created a branch, worked on your code and committed your changes to your local git repository you are ready to create a review request.
- Go to the Github Web interface, find your fork, switch to the branch you have been working on and create a pull request (there should be a button for it).
-
Enter an appropriate "Title".
-
In the "Write" field, be sure to include the following:
- Issue number in the following format #XXX
- Summary of the issue and your solution to the issue
- What testing was performed
In general, after a review-request has been uploaded, there are 3 steps code, topic of a review-request, has to go through before it gets checked in into the MarkUs source code repository: 1. comment on review-request 2. revise review-request (if need be) and 3. check in changed source code (provided the review-request got a green light). Steps one and two need to be repeated as appropriate until the review-request gets an ok (or gets dropped without being committed at all).
But how to comment review-requests? The following steps outline how to do these things:
-
Go to https://github.com/MarkUsProject/Markus/pulls/ and log in with your credentials.
-
Once you have the desired review-request open, click on "Files Changed".
-
Comment code by clicking the blue + on the on appropriate line numbers on the left. By pressing "Comment on this line", your review comments will be published immediately.
-
If you think code is ready to be committed, give a message like "Good to Commit" or "Ready for Merge". A core MarkUs developer will merge the pull request.