Отвлекать в мессенджере — мерзко. Повышать культуру работы с почтой, конечно же, намного сложнее.
Бот в мессенджере — как одна из опций. Все люди разные, и по-разному строят свою работу. Коллеги, которым удобнее получать сообщения по почте, могут даже не включать оповещения в мессенджере.
Некоторые просматривают PRы непосредственно через GitHub в определенный промежуток времени своего рабочего дня — это нормально. Потому и SLA у нас — один рабочий день.
Как поменялось качество ревью?
Качество ревью довольно сложно измерить. По нашим меркам качество ревью не изменилось. Также мы стараемся по максимуму автоматизировать всевозможные проверки, чтобы ревьюеры фокусировались исключительно на самом важном.
Так и есть: абсолютно все могут делать ревью. Ревьюшница случайно выбирает двух человек из всей команды. Причем она равномерно распределяет нагрузку по всем.
И даже если ревьюшница не призвала в PR — есть возможность провести ревью на добровольных началах.
У нас в команде не принято создавать PR на «10 экранов». Мы стараемся декомпозировать задачи так, чтобы их возможно было и быстро кодить, и ревьюить, и тестировать.
Что по поводу 24 часов (а точнее 9h) — это договоренность, которой мы достигли всей командой. Мы вместе пришли к тому, что нам комфортнее будет проводить код-ревью за данное время.
В то же время, мы не заставляем обязательно укладываться за один день. Цитирую:
Мы всегда с пониманием относимся к пулл-реквестам, в которых идёт живое, полезное обсуждение, невзирая на сбитую планку в один день.
Бот в мессенджере — как одна из опций. Все люди разные, и по-разному строят свою работу. Коллеги, которым удобнее получать сообщения по почте, могут даже не включать оповещения в мессенджере.
Некоторые просматривают PRы непосредственно через GitHub в определенный промежуток времени своего рабочего дня — это нормально. Потому и SLA у нас — один рабочий день.
Качество ревью довольно сложно измерить. По нашим меркам качество ревью не изменилось. Также мы стараемся по максимуму автоматизировать всевозможные проверки, чтобы ревьюеры фокусировались исключительно на самом важном.
И даже если ревьюшница не призвала в PR — есть возможность провести ревью на добровольных началах.
Что по поводу 24 часов (а точнее 9h) — это договоренность, которой мы достигли всей командой. Мы вместе пришли к тому, что нам комфортнее будет проводить код-ревью за данное время.
В то же время, мы не заставляем обязательно укладываться за один день. Цитирую: