Skip to content

Instantly share code, notes, and snippets.

View gretchenfitze's full-sized avatar
👾

Rita Klubochkina gretchenfitze

👾
View GitHub Profile

Keybase proof

I hereby claim:

  • I am gretchenfitze on github.
  • I am gretchenfitze (https://keybase.io/gretchenfitze) on keybase.
  • I have a public key ASAQkzBm5kmqj4wDhgON_sOe17SuAdjLnbIWGAFN2mkN8Qo

To claim this, I am signing this object:

@gretchenfitze
gretchenfitze / review.md
Created May 8, 2018 13:21
Что проверять на code review
  • Ревью - задача с наивысшим приоритетом. Желательно приступать к ревью как можно скорее. Это поможет избежать массового мержа задач в конце спринта и не будет тратить время коллег, особенно если задачи взаимозависимы. Начинаем ревью задачи при первом стабильном билде.
  • Прежде всего нужно проверить соответствие указанного номера задачи, репозитория и версии указанным в Jira, а в ходе ревью обратить внимание, соответствуют ли внесенные изменения поставленной задаче.
  • Названия коммитов и PR пишутся с большой буквы, они должны содержать номер задачи, описывать внесенные изменения и быть обезличенными. Плохой пример: “сделал правки”, хороший пример: “UFSUI-1111 Исправлена работа скролла в Select при открытии вверх”.
  • На ревью обязательно нужно как посмотреть код, так и проверить корректность работы компонента с внесенными изменениями. Код стоит проверять досконально в каждом файле. При проверке работы нужно попытаться воспроизвести все возможные кейсы использования компонента. Не стоит пренебрегать одним из ш