Как оформить pull request, который быстро проверят?
- Что писать в описании pull request?
- Как отвечать на замечания ревью?
Маленький PR (до нескольких сотен строк) с одной задачей. В описании: что сделано и зачем (ссылка на задачу), как проверить (шаги, тесты), на что обратить внимание, скриншоты для интерфейса. Перед отправкой — самопроверка диффа и зелёный CI. На замечания отвечают по существу: исправить или объяснить, почему иначе; не спорить о вкусе. Ветку держат актуальной с main.
- Рефакторинг и новая функция — разными PR
- Черновик (draft) — чтобы получить ранний отзыв на подход
- После исправлений — ответить на каждый комментарий или отметить решённым
А ещё PR на 2000 строк ревьюят поверхностно: «Looks good» без чтения
Подробно
Шаблон описания:
Что: вход по одноразовому коду из SMS.
Зачем: задача AUTH-142, 30% пользователей забывают пароль.
Как проверить: make test; вручную — /login, «Войти по коду».
Внимание: новая таблица login_codes, миграция в 0042.Работа с ревью: благодарить за находки, задавать вопросы, если замечание непонятно, не принимать на свой счёт; спорные вопросы — голосом за пять минут вместо двадцати комментариев.
Что хотят услышать
- Маленький PR с одной целью.
- Описание: что, зачем, как проверить.
- Самопроверка и CI.
- Конструктивная работа с замечаниями.
Проверьте себя
Один верный вариант. Подумайте, потом откройте ответ.
Показать варианты и ответ
- ✓Небольшой PR на одну задачу, описание «что, зачем, как проверить», зелёный CI
- ✗Собрать все изменения за месяц в один PR, чтобы была видна картина
- ✗Описание не нужно — код говорит сам за себя
- ✗Отправлять PR с красным CI, ревьюер разберётся
Разбор темы
- Pull request, код-ревью и стратегии ветвления
Как оформить pull request, который быстро проверят, как ревьюить по существу, чем trunk-based отличается от GitFlow и что делает CI/CD.
Связанные вопросы
Разбор собеседования с sotto
После собеседования sotto разбирает разговор: оценка, каждый вопрос — хорошо, частично или слабо, чего не хватило в ответе и над чем поработать к следующему разу. Первые 30 минут бесплатно.