Двукратный публичный code review
Евгения Фирсова
Code review на двоих
Польза очевидна:
• повышение качества кода;
• устранение ошибок;
• взаимное обучение;
• обмен «тайным...
Чего не хватает для счастья?
Резонно хотеть:
• автору кода:
– как можно больше замечаний по ревью;
– взгляд на код с разны...
Публичный code review
Если участвует вся команда:
• больше активных участников:
– выше качество финального кода;
– быстрая...
Процесс публичного code review
разработка подготовка
запрос
code review:
обсуждение
правки
тестирование
подготовка
запрос
...
Запрос на code review
Состав запроса на code review:
• тип ревью (промежуточный, финальный);
• что сделано
(ссылка на рели...
Обсуждения в процессе code review
Да:
• для автора кода:
– отдельный ответ на каждый комментарий;
– «исправлено» + хэш ком...
Вхождение в процесс
Этапы для автора кода:
1. знакомство с правилами;
2. тотальное «да»;
3. полноценное участие.
Этапы для...
Эффект от публичного code review
В числах:
• выложено 39 релизов за квартал;
• запрошено 54 code review;
• 1..9 ревьюеров ...
Активность ревьюеров
0
2
4
6
8
10
1 2 3 4 5 6 7 8 9
ревьюеры
разы
code review на двоих больше участников
больше замечаний
Уточнения? Вопросы?
saigo@yandex.ru — задать любые вопросы
www.control-freak.ru — почитать подробнее об управлении версиям...
Upcoming SlideShare
Loading in …5
×

Двухкратный публичный code review, Евгения Фирсова (Яндекс)

586 views
453 views

Published on

Published in: Internet
0 Comments
0 Likes
Statistics
Notes
  • Be the first to comment

  • Be the first to like this

No Downloads
Views
Total views
586
On SlideShare
0
From Embeds
0
Number of Embeds
41
Actions
Shares
0
Downloads
5
Comments
0
Likes
0
Embeds 0
No embeds

No notes for slide

Двухкратный публичный code review, Евгения Фирсова (Яндекс)

  1. 1. Двукратный публичный code review Евгения Фирсова
  2. 2. Code review на двоих Польза очевидна: • повышение качества кода; • устранение ошибок; • взаимное обучение; • обмен «тайными знаниями»; • соблюдение code style; • укрепление рабочих связей.
  3. 3. Чего не хватает для счастья? Резонно хотеть: • автору кода: – как можно больше замечаний по ревью; – взгляд на код с разных сторон; • команде: – быть в курсе параллельных изменений.
  4. 4. Публичный code review Если участвует вся команда: • больше активных участников: – выше качество финального кода; – быстрая/ранняя корректировка; – общая база терминов; – проработанный навык формулирования; • больше пассивных участников: – обучение на чужих ошибках (а я так не делаю ли?); – сравнение реализаций (а я такое же не делаю ли?); – плавное вхождение в новые технологии; – информирование через любопытство.
  5. 5. Процесс публичного code review разработка подготовка запрос code review: обсуждение правки тестирование подготовка запрос code review: обсуждение правки релизавтотесты
  6. 6. Запрос на code review Состав запроса на code review: • тип ревью (промежуточный, финальный); • что сделано (ссылка на релиз, список задач, описание требований, …); • где посмотреть (файл c diff’ом, хэш коммита, указание на репозиторий, …); • на что обратить внимание; • ограничения (дедлайн на передачу в тестирование, близкий релиз, хот-фикс, …).
  7. 7. Обсуждения в процессе code review Да: • для автора кода: – отдельный ответ на каждый комментарий; – «исправлено» + хэш коммита; – «не согласен» + аргументация; – «не успеваю» + фиксация задачи; – «спасибо»; • для ревьюера: – проверка выполненных по замечаниям правок; – «спасибо». Нет: • holy war; • переход на личности.
  8. 8. Вхождение в процесс Этапы для автора кода: 1. знакомство с правилами; 2. тотальное «да»; 3. полноценное участие. Этапы для ревьюера: 1. знакомство с правилами; 2. read only; 3. «тихий» режим; 4. выделенная полуформальная задача, например: – соблюдение code style; – вычитка комментариев; 5. полноценное участие.
  9. 9. Эффект от публичного code review В числах: • выложено 39 релизов за квартал; • запрошено 54 code review; • 1..9 ревьюеров (из группы в 9 разработчиков); • получено 405 комментариев; • найдено 8 логических ошибок; • найдено 12 критичных ошибок.
  10. 10. Активность ревьюеров 0 2 4 6 8 10 1 2 3 4 5 6 7 8 9 ревьюеры разы code review на двоих больше участников больше замечаний
  11. 11. Уточнения? Вопросы? saigo@yandex.ru — задать любые вопросы www.control-freak.ru — почитать подробнее об управлении версиями, задачами, проблемами и людьми

×