№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