Artwork

Вміст надано Денис, Ігор, Саша. Весь вміст подкастів, включаючи епізоди, графіку та описи подкастів, завантажується та надається безпосередньо компанією Денис, Ігор, Саша або його партнером по платформі подкастів. Якщо ви вважаєте, що хтось використовує ваш захищений авторським правом твір без вашого дозволу, ви можете виконати процедуру, описану тут https://uk.player.fm/legal.
Player FM - додаток Podcast
Переходьте в офлайн за допомогою програми Player FM !

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

51:51
 
Поширити
 

Manage episode 360770798 series 3361795
Вміст надано Денис, Ігор, Саша. Весь вміст подкастів, включаючи епізоди, графіку та описи подкастів, завантажується та надається безпосередньо компанією Денис, Ігор, Саша або його партнером по платформі подкастів. Якщо ви вважаєте, що хтось використовує ваш захищений авторським правом твір без вашого дозволу, ви можете виконати процедуру, описану тут https://uk.player.fm/legal.

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

Робочі посилання і коментарі в каналі ⁠⁠⁠⁠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://www.streambeats.com/⁠⁠⁠⁠ | ⁠⁠⁠⁠@stasgavrylov

  continue reading

43 епізодів

Artwork
iconПоширити
 
Manage episode 360770798 series 3361795
Вміст надано Денис, Ігор, Саша. Весь вміст подкастів, включаючи епізоди, графіку та описи подкастів, завантажується та надається безпосередньо компанією Денис, Ігор, Саша або його партнером по платформі подкастів. Якщо ви вважаєте, що хтось використовує ваш захищений авторським правом твір без вашого дозволу, ви можете виконати процедуру, описану тут https://uk.player.fm/legal.

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

Робочі посилання і коментарі в каналі ⁠⁠⁠⁠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://www.streambeats.com/⁠⁠⁠⁠ | ⁠⁠⁠⁠@stasgavrylov

  continue reading

43 епізодів

Усі епізоди

×
 
Loading …

Ласкаво просимо до Player FM!

Player FM сканує Інтернет для отримання високоякісних подкастів, щоб ви могли насолоджуватися ними зараз. Це найкращий додаток для подкастів, який працює на Android, iPhone і веб-сторінці. Реєстрація для синхронізації підписок між пристроями.

 

Короткий довідник