Код-ревью На страже ваших багов.

Что это?

«Проверка кода с целью обнаружения и исправления ошибок, которые остались

Зачем?

Повышаем качество кода
Учимся писать код у других
Узнаем новое о коде проекта

Почему не делаем?

Да как-то… ссыкотно

Все увидят баги в моем коде / Мой код

Где брать время?

Код

QA

Код-ревью

Релиз

QA

QA

QA

Код-ревью

Внедрить в процедуру проверки каждой задачи.

Давать эстимейты, учитывая 10% времени на ревью.

Эстимейт

В результате, время окупается за счет качества.

За чьи деньги?

Повышаем качество кода.
Повышаем вовлеченность разработчиков.
Повышаем эффективность разработки и рейт.
Profit!!!

Кто должен делать?

Тех. лид / менеджер
Обычно, опытнее и авторитетнее всех.
2-3

При чем тут Drupal?

Приоритеты ревью

Устранение уязвимостей
Нахождение багов
Исправление форматирования

Что просматриваем?

1. Все PHP-файлы (.module, .inc, .install, .test)
Уязвимости. Баги. Форматирование.
2. Все

Рецепты код-ревью

shvetsgroup.com/ru/blog/code

Спасибо за внимание!