Buff Development
Ревью и публикация

Решение по ревью

Одобрить или отправить на доработку.

Итог ревью — одно из двух решений. Панель решения стоит под списком замечаний (на телефоне она закреплена внизу экрана), и над кнопками написано, что произойдёт и почему кнопка недоступна — ещё до нажатия.

Первая фраза называет адрес публикации: например, «Одобрение создаст PR в GitLab · ats-system/quotes-service». Если исходный репозиторий не подключён, там сказано, что изменения вольются в репозитории проекта.

Одобрить

Публикует изменения в исходные репозитории (PR/MR) и закрывает наши.

Недоступно, пока есть открытые замечания — это ожидает + к исправлению (см. словарь статусов). Кнопка в этом случае выглядит неактивной, а над ней написано, почему: «Одобрить» недоступно: открыты 2 замечания: отправьте на доработку или снимите их».

Два выхода прямо из диалога «Одобрить»:

  • Отклонить все — если замечания не нужны: они уходят в архив, и одобрение продолжается.
  • Отправить на доработку — если по ним нужно переделать (см. ниже).

На доработку

Создаётся подзадача с замечаниями, наши pull request закрываются, разработка перезапускается. Это петля доработки.

  • Есть замечания «к исправлению» — доработка идёт по ним.
  • Замечаний ещё нет — просто опишите, что переделать: комментарий сохранится как замечание и уйдёт в доработку. Отдельно добавлять замечание заранее не нужно.

«Одобрить» недоступно — что делать? Значит, есть открытые замечания. Либо снимите те, что не нужны (в самом замечании — «Снять замечание», или в диалоге «Одобрить» → «Отклонить все»), либо отправьте на доработку — тогда по ним переделают. «К исправлению» означает «войдёт в доработку», а не «готово».

Решение по ревью — «Одобрить» или «На доработку», под списком замечаний

On this page