Будьте благодарны друг другу. ваш код смотрят — тратят время и дают вам фидбек. Если вы смотрите чужой код — его писали, страдали, старались.
Если вы автор PR
- Заботьтесь о ревьюерах. Они тратят время на помощь вам. ****
- Начните с подробного и понятного заголовка для PR
- Опишите изменения и как они устроены, принятые архитектурные решения
- Обозначьте места, в которых вы хотели бы фидбека
- Сообщите какие незакрытые вопросы и беспокойства остались
- До того как снимать WIP:
- Протестируйте свой код
- Проведите сами себе ревью кода
- Убедитесь что все тесты и линтеры проходят
- Задавайте вопросы
- Стремитесь иметь как можно меньшие по размеру PRы. Больше 300 строк — недопустимо.
Если вы ревьюер
- Будьте вежливы и добры, вы смотрите на труд другого человека
- Найдите чему вы можете научиться из этого кода
- Задавайте вопросы вместо критики
- Говорите конкретно вместо абстракций
- Предлагайте улучшения вместо деконструкции решения
- Вместо спора кооперируйте
- Цель — улучшить наш код и команду
О чем мы паримся
https://hub.github.com