← все записи

Я не верю в code review

👁 393💬 5
Кто-то должен был это сказать. Поэтому я решил стать первым. Проблемы code review: 1. Замедляет Фича готова -> ждём ревью 1-2 дня -> правим -> ещё ревью 2. Nitpicking 90% комментов: "тут пробел лишний", "переименуй в camelCase", "удали/добавь комменты" 3. Велосипеды Кто-то: "Я бы сделал по-другому" и понеслось, споры - споры - споры 4. False security Код прошёл ревью != безопасность. Потому что баги всё равно будут. Что работает намного лучше: 1. Pair programming (когда нужно) 2. Автотесты (находят больше багов) 3. Code Style и Linters (автоматизируют nitpicking) 4. Post-merge review to master (асинхронно, без блокировки) Возможно я просто никогда не видел нормальных процессов с Code review. Наверное это как раз в корпорациях сделано грамотно. А в стартапах скорость важнее идеального кода. 9/100 Telegram
143

Кирилл Гаврилов

Подписаться