# Code Review

# Для создающих

  • Code Review может занимать до 2х суток. Если на вторые сутки вы не получили реакции, зовите нужных людей в рамка утреннего "MR Digest".
  • Относитесь к ревью как к дополнительному обучению. Ваши коллеги помогут показать места для улучшения и ошибки. В целом качество кода будет повышаться.
  • Не ассоциируйте себя с кодом. Комментарии коллег относятся к коду, не нужно воспринимать это как личную критику.
  • Если вы встретили сообщение с Nit: (Nitpicking) - это значит, что прнимать пожелание или нет - это на ваше усмотрение.

# Для проверяющих

  • Начинайте день с проверки текущих MergeRequest и заглядывайте в свободное время, чтобы посмотреть.
  • Сначала пройдитесь быстро взглядом по изменениям, затем еще раз более детально.
  • Старайтесь избегать прямых указаний, задавайте конкретные вопросы. Коллега может не иметь вашего опыта и может не понять пространственных вопросов.
  • Если нашли опечатку, лучше исправьте ее сами. Нажмите на иконку карандраша и внесите изменения.
  • Не забудьте нажать на Finish Code Review после того, как закончили ревью.
  • Если все OK, ставьте Approve.
  • Начинайте сообщение с Nit: (от слова Nitpicking) для случаев, когда вы хотите дать пожелания, но делать или нет оставляете на усмотрение автора.
  • Если заданный вами вопрос решен, нажмите Mark as Resolved. Тот кто оставил комментарий должен следить за его завершением.

# Ссылки

  • https://google.github.io/eng-practices/review/
Last Updated: 2/12/2021, 10:55:27 AM