Улучшаем код-ревью: чек-лист и примеры (Сергей Жук, Skyeng)

preview_player
Показать описание
Как команда мобильного бэкенда Skyeng оптимизировала процесс код-ревью и какие стратегии могут сработать у вас.

1:13 Типичные проблемы "жирных" пул-реквестов
4:51 На какие категории изменений разбивать реквест, как их ревьюить
13:16 Упрощаем реквест на примере
21:08 Чек-лист хорошего пул-реквеста
22:40 "Зачем так заморачиваться?" или Какие выгоды получите вы и команда
30:40 Вопросы докладчику
Рекомендации по теме
Комментарии
Автор

1:13 - Типичные проблемы "жирных" пул-реквестов
4:51 - На какие категории изменений разбивать реквест, как их ревьюить
13:16 - Упрощаем реквест на примере
21:08 - Чек-лист хорошего пул-реквеста
22:40 - "Зачем так заморачиваться?" или Какие выгоды получите вы и команда
30:40 - Вопросы докладчику

SkyengITeam
Автор

Спасибо за доклад.
На самом деле удивительно, что такая строгая дисциплина оформления PR может поддерживаться всеми участниками на протяжении длительного времени.
И не понял почему названия пулл реквестов в SkyEng на русском. Хотя месседж коммитов и комментарии в коде на английском.

isfland
Автор

В разделе вопросов-ответов ребята указывают на очевидную проблему с конфликтами, и возможностью улучшить подход: сделать категории для коммитов и держать коммиты одной группы рядом с друг-другом (группировать можно префиксом, вроде `Refactor: ...`, 'Code Style: ...', и т.д.) вместо использования группы отдельных (или наследуемых друг от друга) пул-реквестов. Ну и очевидно, что изменения в code style вообще в идеале должны НЕ быть частью пул-реквеста, а оформляться автоматом в процессе CI/CD.

andrey.bocharov
Автор

Вопросы:
1. Разве все эти вещи, небольшие PR, отсутствие разных контекстов внутри него (когда не смешиваем рефакторинг, фичи) и т.д. в enterprise разработке не являются просто здравым смыслом? На мой взгляд, понимание того, что здесь сказано, приходит довольно быстро и достаточно сказать команде об этом один раз и дальше подобных вещей не наблюдается.
2. И второй вопрос - не является ли большой PR из-за крупной фичи изначально следствием неправильно поставленной задачи (например, не смогли правильно декомпозировать задачу от бизнеса), по вашему опыту?

ВалентинСпиридонов-см
Автор

Сколько по времени у вас проходят пуллл реквест в среднем?

twitatalex
Автор

1. Является ли это регламентом внутри команды? Т.е. обязательно ли это для всех?
2. Для всех ли размеров диффа вы используете такой подход? Заранее не всегда известен размер диффа для начального разработчика системы.

petrosnn