Слайд 2Что это?
«Проверка кода с целью обнаружения и исправления ошибок, которые остались незамеченными
![Что это? «Проверка кода с целью обнаружения и исправления ошибок, которые остались](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-1.jpg)
в начальной фазе разработки»
Слайд 3Зачем?
Повышаем качество кода
Учимся писать код у других
Узнаем новое о коде проекта
![Зачем? Повышаем качество кода Учимся писать код у других Узнаем новое о коде проекта](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-2.jpg)
Слайд 5Да как-то… ссыкотно
Все увидят баги в моем коде / Мой код идеален!
Нету
![Да как-то… ссыкотно Все увидят баги в моем коде / Мой код](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-4.jpg)
времени.
Клиент будет недоволен, что мы тратим бюджет на проверку качества кода, который и так должен быть идеален.
Слайд 7Код
QA
Код-ревью
Релиз
QA
QA
QA
Код-ревью
Внедрить в процедуру проверки каждой задачи.
![Код QA Код-ревью Релиз QA QA QA Код-ревью Внедрить в процедуру проверки каждой задачи.](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-6.jpg)
Слайд 8Давать эстимейты, учитывая 10% времени
на ревью.
Эстимейт
![Давать эстимейты, учитывая 10% времени на ревью. Эстимейт](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-7.jpg)
Слайд 9В результате, время окупается за счет качества.
![В результате, время окупается за счет качества.](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-8.jpg)
Слайд 10За чьи деньги?
Повышаем качество кода.
Повышаем вовлеченность разработчиков.
Повышаем эффективность разработки и рейт.
Profit!!!
![За чьи деньги? Повышаем качество кода. Повышаем вовлеченность разработчиков. Повышаем эффективность разработки и рейт. Profit!!!](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-9.jpg)
Слайд 11Кто должен делать?
Тех. лид / менеджер
Обычно, опытнее и авторитетнее всех.
2-3 других
![Кто должен делать? Тех. лид / менеджер Обычно, опытнее и авторитетнее всех.](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-10.jpg)
разработчиков
Две неопытные головы лучше одной неопытной.
Слайд 13Приоритеты ревью
Устранение уязвимостей
Нахождение багов
Исправление форматирования
![Приоритеты ревью Устранение уязвимостей Нахождение багов Исправление форматирования](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-12.jpg)
Слайд 14Что просматриваем?
1. Все PHP-файлы (.module, .inc, .install, .test)
Уязвимости. Баги. Форматирование.
2. Все JS-файлы
Баги.
![Что просматриваем? 1. Все PHP-файлы (.module, .inc, .install, .test) Уязвимости. Баги. Форматирование.](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-13.jpg)
Не использование Drupal JS и JQuery. Форматирование.
3. Все CSS-файлы
Ошибки форматирования.
Слайд 15Рецепты код-ревью
shvetsgroup.com/ru/blog/code
![Рецепты код-ревью shvetsgroup.com/ru/blog/code](/_ipx/f_webp&q_80&fit_contain&s_1440x1080/imagesDir/jpg/360398/slide-14.jpg)