№36: Код рев'ю – для чого, і як (не) робити?

Опівночні Балачки

🔞 Тут будуть матюки 🔞

Робочі посилання і коментарі в каналі ⁠⁠⁠⁠https://t.me/midnight_chatter

• 0:01-3:10 Інтро. Наші перші код-рев’ю.
• 3:11-5:43 Чи є code review обовʼязковою практикою?
• 5:44-9:50 Навіщо робити код рев’ю: обмінюємося знаннями. Життя буденне при дистанційній роботі. Pull request як арена для молодих челенджити старі шляхи написання коду
• 9:51-11:20 Навіщо №2: шукаємо дефекти і налагоджуємо стиль, рев’ювлємо дизайн рішення
• 11:21-12:28 Що є результатом код рев’ю? Хто мусить мерджити пулл реквест?
• 12:29-12:49 Навіщо №3: шаримо відповідальність
• 12:50-13:24 Навіщо №4: покращуємо здатність естімейтити в майбутньому
• 13:25-15:18 Гітхабівські “пулл реквести” (PR) проти Гітлабівських “мердж реквестів” (MR)
• 15:19-17:05 Недолік рев’ю “не всі дефекти знаходять”. Google про code review у статті (але про 1/6 була брехня, Сашко перепрошує, знаходять “скільки знаходиться”, якщо говорити про стати, хіба у статті 2014 від MS було про “в середньому 4 дефекта в рев’ю”, але там все складніше і в ноутсах не перекажемо)
• 17:06-18:39 Ще недолік: довше закриваємо таски, що з цим робити?
• 18:40-20:57 Вирішуємо проблему код рев’юверів “не розумію, про що ця зміна?”
• 20:58-22:34 Ділимо фідбек на важливий і неважливий, а також питання. А також про комунікацію англійською.
• 22:35-23:57 Золоте правило рев’ю “Не задовбуйте”. Тудушки і тікети “на потім”. Допомагайте молодим
• 23:58-26:22 Як імплементувати цей поділ на популярних платформах для код рев’ю
• 26:23-29:11 А ще, можна коментарями надавати контекст. Ну і робіть самі собі code review
• 29:12-31:15 Якщо хтось створив і змерджив PR на вихідних – чи треба його пост-фактум рев’ювити?
• 31:16-35:59 На що найбільше спрямовувати увагу при код-рев’ю?
• 36:00-37:50 Чи є прохання написати тести порушенням заповіді “не задобвуй”? Без тестів – нікуди
• 37:51-41:34 Скільки рев’юверів треба на 1 пулл реквест? Знову посилаємося на статті від Google & MS. Як працюють CODEOWNERS у Github
• 41:35-43:38 Галопом по Європах: автоматизуємо тривіальне, де рев’ювити дизайн рішення і як довжина ПР впливає на тривалість рев’ю. Про кількість рядків за годину рев’ю посилаємося на smartbear
• 43:41-45:06 І взагалі, не напружуйтесь. Бот експірієнс інженери ніколи не думають про перфокарти
• 45:07-45:36 Як зменшити кількість рев’ю? Парне програмування. Рев’ю сесії
• 45:37-46:18 Gitlab і їх рулетка для рев’юверів (а також хай тут буде їх матеріали про код рев’ю)
• 46:19-47:50 Чи можна було б жити без код рев’ю завжди? Тести, ШІ і nocode
• 47:51-50:10 Не забувайте, що з того боку – жива людина
• 50:11-51:51 Outro, робіть код рев’ю цьому випуску і не будьте токсіком. Наступного тижня відпочиваємо

Долучайтесь до наших соцмереж:

• ⁠⁠⁠⁠https://t.me/midnight_chatter⁠⁠⁠⁠
• Twitter ⁠⁠⁠⁠@O_Balachky⁠⁠⁠⁠
• TikTok ⁠⁠⁠⁠@o_balachky⁠⁠⁠⁠

Музика: ⁠⁠⁠⁠https://www.streambeats.com/⁠⁠⁠⁠ | ⁠⁠⁠⁠@stasgavrylov