Re:Ask HN: How to peer review when we're both new to the code?
by Juliate on Thursday February 07, 2019 @04:16AM
(#19104024)
Draw a map on a whiteboard, to build and share your understanding of the code base and the context as you progress together reviewing each other.
If the reviews are too large/complex for you to grasp, that may mean that:
* either it's not focused on one specific topic, so it should be split up,
* either it's actually a more complex issue you'll need to dive together in to grasp it (back to the drawing board).
Depending on the code base:
* if you can plug a cross reference tool (such as https://lxr.sourceforge.io/ or anything that suits you), that will help to jump from one place to an other
* if there are tests (hopefully), check if you have a test coverage report (or set it up). That's good info to have too.
If the reviews are too large/complex for you to grasp, that may mean that: * either it's not focused on one specific topic, so it should be split up, * either it's actually a more complex issue you'll need to dive together in to grasp it (back to the drawing board).
Depending on the code base: * if you can plug a cross reference tool (such as https://lxr.sourceforge.io/ or anything that suits you), that will help to jump from one place to an other * if there are tests (hopefully), check if you have a test coverage report (or set it up). That's good info to have too.