Чеклист Кевина Лондона: на какие детали архитектуры и стиля стоит обращать внимание во время код-ревью.

Советы по обсуждению кода:

• Задавайте вопросы: «Как работает эта функция?», «Что придется передалать при изменении требований?».
• Хвалите за хорошие решения.
• Встретьтесь с автором, чтобы обсудить детали, если чувствуете, что обмен комментариями займет (или уже занимает) много времени.
• Объясняйте свой ход мысли, когда предлагаете что-то изменить. Тогда ваши предложения не покажутся пустыми придирками.
• Указывайте важность. Явно отмечайте, по каким из ваших замечаний требуется внести изменения. По остальным дайте автору возможность принять решение самостоятельно.