Код-ревью в Drupal
- 1. Код-ревьюНа страже ваших багов.© Александр Швец
- 2. Что это?«Проверка кода с целью обнаружения и исправления ошибок, которые остались незамеченными в начальной фазе разработки»
- 3. Зачем?Повышаем качество кодаУчимся писать код у другихУзнаем новое о коде проекта
- 5. Да как-то… ссыкотноВсе увидят баги в моем коде / Мой код идеален!Нету времени.Клиент будет недоволен, что мы тратим бюджет на проверку качества кода, который и так должен быть идеален.
- 7. Внедрить в процедуру проверки каждой задачи.QAКод-ревьюКодQAКод-ревьюРелизQAQA
- 8. Давать эстимейты, учитывая 10% временина ревью.Эстимейт
- 9. В результате, время окупается за счет качества.
- 10. За чьи деньги?Повышаем качество кода.Повышаем вовлеченность разработчиков.Повышаем эффективность разработки и рейт.Profit!!!
- 11. Кто должен делать?Тех. лид/ менеджер Обычно, опытнее и авторитетнее всех.2-3 других разработчиков Две неопытные головы лучше одной неопытной.
- 12. При чем тут Drupal?
- 13. Приоритеты ревьюУстранение уязвимостейНахождение баговИсправление форматирования
- 14. Что просматриваем?1. Все PHP-файлы (.module, .inc, .install, .test)Уязвимости.Баги. Форматирование.2. Все JS-файлыБаги.Не использование Drupal JS и JQuery. Форматирование.3. Все CSS-файлыОшибки форматирования.
- 17. Александр ШвецShvetsGroup,Директор Почта: [email protected]Телефон:+38 (097) 6833155Skype:neo.chief