From 2098931e3d13ebb2cb5bc7c7293f5413ea7b50f6 Mon Sep 17 00:00:00 2001 From: GeckoEidechse Date: Tue, 19 Jul 2022 21:36:12 +0000 Subject: GitBook: [#56] Add section about PR reviews --- docs/.gitbook/assets/review1.png | Bin 0 -> 15941 bytes docs/.gitbook/assets/review2.PNG | Bin 0 -> 37943 bytes docs/.gitbook/assets/review3.png | Bin 0 -> 15790 bytes docs/.gitbook/assets/review4.png | Bin 0 -> 26222 bytes docs/SUMMARY.md | 1 + .../development/reviewing.md | 60 +++++++++++++++++++++ 6 files changed, 61 insertions(+) create mode 100644 docs/.gitbook/assets/review1.png create mode 100644 docs/.gitbook/assets/review2.PNG create mode 100644 docs/.gitbook/assets/review3.png create mode 100644 docs/.gitbook/assets/review4.png create mode 100644 docs/modding-and-development/development/reviewing.md diff --git a/docs/.gitbook/assets/review1.png b/docs/.gitbook/assets/review1.png new file mode 100644 index 0000000..f5d37a2 Binary files /dev/null and b/docs/.gitbook/assets/review1.png differ diff --git a/docs/.gitbook/assets/review2.PNG b/docs/.gitbook/assets/review2.PNG new file mode 100644 index 0000000..8d12f4d Binary files /dev/null and b/docs/.gitbook/assets/review2.PNG differ diff --git a/docs/.gitbook/assets/review3.png b/docs/.gitbook/assets/review3.png new file mode 100644 index 0000000..cc6de54 Binary files /dev/null and b/docs/.gitbook/assets/review3.png differ diff --git a/docs/.gitbook/assets/review4.png b/docs/.gitbook/assets/review4.png new file mode 100644 index 0000000..68a7ef4 Binary files /dev/null and b/docs/.gitbook/assets/review4.png differ 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. + +![](../../.gitbook/assets/review1.png) + +From there you can select the line(s) you want to leave a comment on. + +![](../../.gitbook/assets/review2.PNG) + +Type in your comment and click on _"Start a review"_. + +![](../../.gitbook/assets/review3.png) + +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. + +![](../../.gitbook/assets/review4.png) + +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 -- cgit v1.2.3