diff options
| -rw-r--r-- | docs/.gitbook/assets/review1.png | bin | 0 -> 15941 bytes | |||
| -rw-r--r-- | docs/.gitbook/assets/review2.PNG | bin | 0 -> 37943 bytes | |||
| -rw-r--r-- | docs/.gitbook/assets/review3.png | bin | 0 -> 15790 bytes | |||
| -rw-r--r-- | docs/.gitbook/assets/review4.png | bin | 0 -> 26222 bytes | |||
| -rw-r--r-- | docs/SUMMARY.md | 1 | ||||
| -rw-r--r-- | docs/modding-and-development/development/reviewing.md | 60 |
6 files changed, 61 insertions, 0 deletions
diff --git a/docs/.gitbook/assets/review1.png b/docs/.gitbook/assets/review1.png Binary files differnew file mode 100644 index 0000000..f5d37a2 --- /dev/null +++ b/docs/.gitbook/assets/review1.png diff --git a/docs/.gitbook/assets/review2.PNG b/docs/.gitbook/assets/review2.PNG Binary files differnew file mode 100644 index 0000000..8d12f4d --- /dev/null +++ b/docs/.gitbook/assets/review2.PNG diff --git a/docs/.gitbook/assets/review3.png b/docs/.gitbook/assets/review3.png Binary files differnew file mode 100644 index 0000000..cc6de54 --- /dev/null +++ b/docs/.gitbook/assets/review3.png diff --git a/docs/.gitbook/assets/review4.png b/docs/.gitbook/assets/review4.png Binary files differnew file mode 100644 index 0000000..68a7ef4 --- /dev/null +++ b/docs/.gitbook/assets/review4.png diff --git a/docs/SUMMARY.md b/docs/SUMMARY.md index 1569fc5..149bf54 100644 --- a/docs/SUMMARY.md +++ b/docs/SUMMARY.md @@ -43,6 +43,7 @@ * [Deploy](modding-and-development/development/northstarmasterserver/deploy.md) * [Contributing code to Northstar](modding-and-development/development/contributing-code-to-northstar.md) * [Testing](modding-and-development/testing.md) + * [Reviewing](modding-and-development/development/reviewing.md) * [Releases](modding-and-development/development/releases.md) ## Other diff --git a/docs/modding-and-development/development/reviewing.md b/docs/modding-and-development/development/reviewing.md new file mode 100644 index 0000000..c17bba2 --- /dev/null +++ b/docs/modding-and-development/development/reviewing.md @@ -0,0 +1,60 @@ +--- +description: >- + This pages describes how to review PRs to ensure quick and efficient merging. + It is written for both novices and people with experience with GitHub and code + review in general. +--- + +# Reviewing + +For new pull requests to be merged into any of the Northstar repos, they need to reviewed to reduce the changes of introducing bugs as well as ensuring that in case of a feature it's something the community actually wants. + +### Process + +For a review there's two major parts **code review** and **testing**. + +It's not necessary to do both when reviewing as long as you mention what you did when leaving the review comment on GitHub. + +#### Code review + +For code review head to the _"Files changed"_ section of the PR. + + + +From there you can select the line(s) you want to leave a comment on. + + + +Type in your comment and click on _"Start a review"_. + + + +Note that this will **NOT** post your comment immediately! Add any remaining comments to other lines of code, then head to the _Finishing up_ section of this wiki page to see how to post your code review. + + + +#### Testing + +For testing a PR, refer to the following page + +{% content-ref url="../testing.md" %} +[testing.md](../testing.md) +{% endcontent-ref %} + +The TL;DR is to test the aspect that has been changed. + +#### Finishing up + +After you performed the testing and/or code review leave I final comment by clicking on _"Finish your review"_ on the top right of the _"Files changed"_ page of the PR. + + + +In your final comment make sure to mention what you did, e.g. which aspects of the change you tested for or what you considered during code review (format, edge cases, ...). The more detailed this part is the easier it is for other reviewers to tell which aspects they can skip during reviewing to speed up the process. + +After leaving your comment, select the type of feedback. + +* **Comment**: General remarks, use if neither of the later two applies. +* **Approve**: You found no issues when looking at the code or during testing. +* **Request changes**: Either something in the code doesn't look right or you found bugs during testing. + +Finally, click on _"Submit review"_. Your review is now publicly visible, congrats! :D |
