Your SlideShare is downloading. ×
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Двухкратный публичный code review, Евгения Фирсова (Яндекс)
Upcoming SlideShare
Loading in...5
×

Thanks for flagging this SlideShare!

Oops! An error has occurred.

×
Saving this for later? Get the SlideShare app to save on your phone or tablet. Read anywhere, anytime – even offline.
Text the download link to your phone
Standard text messaging rates apply

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

257

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
257
On Slideshare
0
From Embeds
0
Number of Embeds
3
Actions
Shares
0
Downloads
3
Comments
0
Likes
0
Embeds 0
No embeds

Report content
Flagged as inappropriate Flag as inappropriate
Flag as inappropriate

Select your reason for flagging this presentation as inappropriate.

Cancel
No notes for slide

Transcript

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

×