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

2:12:00
 
Поделиться
 

Manage episode 267683293 series 58248
Сделано Konstantin Burkalev и найдено благодаря Player FM и нашему сообществу. Авторские права принадлежат издателю, а не Player FM, и аудиоматериалы транслируются прямо с его сервера. Нажмите на кнопку Подписаться, чтобы следить за обновлениями через Player FM или скопируйте и вставьте ссылку на канал в другое приложение для подкастов.
Товарищи, в этот раз вас ждёт необычный выпуск подкаста! Почему? — Этот выпуск был записан в прямом эфире с обсуждением вопросов зрителей, онлайн-голосованиями и другими активностями. Но не переживайте, весь контент доступен для восприятия в аудио-формате как и прежде! Итак, выпуск посвящён непростой и спорной теме код-ревью. В этом выпуске, помимо меня приняли участие: * Александр Макаров, лид проекта 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), где можно обсудить выпуски, предложить гостей и высказать свои замечания и пожелания!

123 эпизодов