У нас есть негласное правило, что пул реквест должен хорошо выглядит в веб-интерфейсе гитхаба. Мне кажется это отличное соглашение, благодаря которому вводится несколько компромиссов, из-за которых иначе могут возникать споры.

Для начала ширина кода фиксируется на 140 символов, чтобы не было горизонтального скролла. Далее устанавливается очень разумное ограничение на количество одновременных изменений. Я не изучал точно как работает алгоритм гитхаба, но при попытке изменить слишком много файлов одновременно, он покажет только часть. Также не смотрим пул реквесты, которые не мержатся автоматически в мастер или у которых тесты красные (мы работаем по гитхаб-флоу и положительное ревью означает, что код может оказаться на продакшене через несколько минут, поэтому важно чтобы он был готов).

Более глубоким эффектом оказывается разделение рефакторинга и нового функционала. Если например меняется отступ в больших массивах кода, а это часто бывает при рефакторинге, то будет много красного и много зеленого. И уже не хочется в эти изменения что-то добавлять еще, так как пул реквест станет не читаемым. Поэтому все массивные изменения всегда идут отдельными пул реквестами, что сильно уменьшает влияние человеческого фактора.

Пожалуй самое забавное, что через некоторое время формируется ощущение, что полностью зеленые или полностью красные пул реквесты, более безопасные, чем полосатые. К полосатым - всегда повышенное внимание.

Tweet