Несколько команд в моей компании практикуют рабочий процесс проверки кода, который я никогда раньше не видел. Я пытаюсь понять мышление, стоящее за этим, с мыслью, что есть смысл в том, чтобы сделать всю компанию последовательной. (Я участвую в создании нескольких кодовых баз, и меня смутили различия в прошлом.)
- Автор кода отправляет запрос на удаление
- Рецензент проверяет код
- Если рецензент одобряет, он оставляет комментарий в духе «Хорошо выглядит, не стесняйтесь сливаться»
- Если у рецензента есть проблемы, они оставляют комментарий типа «Пожалуйста, исправьте незначительные проблемы X и Y, затем объедините» (для серьезных изменений вернитесь к шагу 2)
- Автор кода вносит изменения в случае необходимости, а затем объединяет его или ее собственный запрос извлечения
У меня есть следующие проблемы:
В случае утверждения на шаге 3 этот рабочий процесс создает, казалось бы, ненужную обратную передачу для автора запроса на извлечение. Рецензент, который уже просматривает код, может сразу же объединить его.
В случае изменений, запрошенных на шаге 3, агентство по объединению запроса на получение ответа теперь остается исключительно за автором PR. Никто кроме автора не будет смотреть на изменения до слияния.
Каковы некоторые другие преимущества или недостатки этого рабочего процесса? Распространен ли этот рабочий процесс в других командах разработчиков?