Код-ревью в drupal
TRANSCRIPT
![Page 1: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/1.jpg)
Код-ревьюНа страже ваших багов.
© Александр Швец
![Page 2: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/2.jpg)
Что это?
«Проверка кода с целью обнаружения и исправления ошибок, которые остались незамеченными в начальной фазе разработки»
![Page 3: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/3.jpg)
Зачем?• Повышаем качество кода• Учимся писать код у других• Узнаем новое о коде проекта
![Page 4: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/4.jpg)
Почему не делаем?
![Page 5: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/5.jpg)
Да как-то… ссыкотно
1. Все увидят баги в моем коде / Мой код идеален!
2. Нету времени.
3. Клиент будет недоволен, что мы тратим бюджет на проверку качества кода, который и так должен быть идеален.
![Page 6: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/6.jpg)
Где брать время?
![Page 7: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/7.jpg)
Код QAКод-ревь
юРелизQAQAQA
Код-ревь
ю
Внедрить в процедуру проверки
каждой задачи.
![Page 8: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/8.jpg)
Давать эстимейты, учитывая 10%
временина ревью.
Эстимейт
![Page 9: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/9.jpg)
В результате, время окупается за счет
качества.
![Page 10: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/10.jpg)
За чьи деньги?
1. Повышаем качество кода.
2. Повышаем вовлеченность разработчиков.
3. Повышаем эффективность разработки и рейт.
4. Profit!!!
![Page 11: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/11.jpg)
Кто должен делать?
• Тех. лид / менеджер Обычно, опытнее и авторитетнее всех.
• 2-3 других разработчиков Две неопытные головы лучше одной неопытной.
![Page 12: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/12.jpg)
При чем тут Drupal?
![Page 13: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/13.jpg)
Приоритеты ревью
1. Устранение уязвимостей2. Нахождение багов3. Исправление форматирования
![Page 14: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/14.jpg)
Что просматриваем?
1. Все PHP-файлы (.module, .inc, .install, .test)Уязвимости. Баги. Форматирование.
2. Все JS-файлыБаги. Не использование Drupal JS и JQuery. Форматирование.
3. Все CSS-файлыОшибки форматирования.
![Page 15: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/15.jpg)
Рецепты код-ревью
shvetsgroup.com/ru/blog/code
![Page 16: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/16.jpg)
Спасибо за внимание!
![Page 17: Код-ревью в Drupal](https://reader034.vdocuments.site/reader034/viewer/2022052600/557f2049d8b42aea318b4f41/html5/thumbnails/17.jpg)
Александр Швец
ShvetsGroup, Директор
Почта: [email protected]Телефон: +38 (097) 6833155 Skype: neo.chief