3 ms·
(non-native english speaker, be kind) I'm not even a good code reviewer but I think I can share some insights: 1 - First of all remember these two things: "The
by kuresto 8y ago
(non-native english speaker, be kind) I'm not even a good code reviewer but I think I can share some insights:
1 - First of all remember these two things: "There is no perfect code" and "There is a human, who doesn't know everything, on the other side".
2 - Understand the scope of the Code. What is this piece trying to achieve?
3 - Read. Calmly.
4 - Read. Calmly. Again. Reading code is hard and takes practice.
5 - If you think something could be done better but it is not required, open the discussion, not everything is a 'change request'.
6 - If you don't understand something, ask.
7 - It's a team play.
8 - Praise what needs to be praised.
9 - Read the tests, for god sake. Try to think test cases that the PR didn't cover.
10 - Understand the moment that the project is. Sometimes some changes are too big for this review. If this is the case, could it be done later?
I know these are pretty simple insights, and no real technique, but they are often forgotten :/
- kuresto 8y agoAh, and last: Don't be afraid.