SDCast #121: Круглый стол про код-ревью

Товарищи, в этот раз вас ждёт необычный выпуск подкаста! Почему? — Этот выпуск был записан в прямом эфире с обсуждением вопросов зрителей, онлайн-голосованиями и другими активностями. Но не переживайте, весь контент доступен для восприятия в аудио-формате как и прежде!

Итак, выпуск посвящён непростой и спорной теме код-ревью. В этом выпуске, помимо меня приняли участие:

* Александр Макаров, лид проекта Yii Framework

* Антон Морев, основатель и СТО Wormsoft

* Сергей Жук, разработчик из SkyEng

Вот далеко не полный перечень тем, которые мы обсудили:

* Каким должно быть идеальное код-ревью?

* Есть ли отличия в процессах код-ревью для крупных компаний, заказной разработки и open source проектов?

* Размеры одного пулл-реквеста: что делать с огромными пул-реквестами?

* Сколько времени должно уходить на ревью одного запроса? Когда и как делать код-ревью?

* Как экономить время на код-ревью?

* Инструменты, облегчающие и помогающие код-ревью

* Человеческий фактор в код-ревью: как не убить критикой все старания?

В процессе беседы мы много вспоминали и приводили примеры из собственного опыта, команд и проектов. Поэтому попытались не просто порассуждать о идеальном ревью, а привести примеры, подходы и практики, работающие в конкретных ситуациях и командах.

Так же за время беседы мы запустили несколько голосований по тематике код-ревью, а в конце дискуссии подвели итоги.

Ссылки на ресурсы по темам выпуска:

* Доклада Сереги Жука: «Code review: быстро и эффективно» (https://www.youtube.com/watch?v=se3F83O0ZiY&feature=youtu.be&t=349) и его текстовая версия (https://habr.com/ru/company/skyeng/blog/443402/)

* Инструменты:

* _https://www.sqlstyle.guide_ (https://www.sqlstyle.guide/) - Руководство по стилю SQL

* _https://prettier.io/_ (https://prettier.io/). An opinionated code formatter

* *conventional: *comments (https://conventionalcomments.org/). Подход к комментированию пулл-реквестов от Пола Слоутера из GitLab

* Статья «Prettier в крупных проектах: тратим 20 минут на настройку, забываем о форматировании навсегда» (https://habr.com/ru/company/skyeng/blog/484992/)

* Гайдлайны код-ревью проекта Yii (https://github.com/yiisoft/docs/blob/master/013-code-review.md)

* Анонс нашего эфира на хабре (https://habr.com/ru/company/skyeng/blog/508148/)

Понравился выпуск? — Поддержи подкаст на patreon.com/KSDaemon (https://www.patreon.com/KSDaemon), звёздочками в iTunes (https://podcasts.apple.com/ru/podcast/software-development-podcast/id890468606?l=en), а так же ретвитом или постом! Заходи в телеграм-чат SDCast (https://t.me/SDCast), где можно обсудить выпуски, предложить гостей и высказать свои замечания и пожелания!