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

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

Apr 16 2023 • 51 mins

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

Робочі посилання і коментарі в каналі ⁠⁠⁠⁠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, робіть код рев’ю цьому випуску і не будьте токсіком. Наступного тижня відпочиваємо