Code Reviews
A code review is the process where team members check each other's code before it is definitively merged. This is important to discover errors or suboptimal code in time and to ensure that everyone adheres to the agreed code standards (think of style, structure, documentation). Additionally, code reviews facilitate knowledge sharing: developers learn from each other's solutions and become familiar with all parts of the codebase, so that not one person is the only one who understands a particular piece of code.
Starting Points
Key Points
- All code is provided with peer feedback using merge requests.
- Consideration is given to the amount of code for which feedback is requested (less is more).
- The feedback during code reviews is constructive and substantive.
- You incorporate received feedback into your code. It is demonstrable that after review round(s) the code has been adjusted or that the student has responded to the comments with arguments. All of this is visible in review comments or commit history.