Як робити рев'ю pull request на GitHub: гайд для PM

Alex Tutor

День рев'ю #134. Перш ніж відкривати бодай один файл: що саме ти перевіряєш?

Ви

Не код. Мабуть, поведінку?

Alex Tutor

Зміну такою, якою її зустріне клієнт. Чи робить вона те, про що просило issue? Чи добре читаються тексти? Чи не робить вона чогось, про що issue не просило? Що буде у випадках, які ніхто не записав?

Daniel

А код лиши інженерам: назви, структуру, швидкодію, який підхід мені варто було обрати. Цим займається Oscar. Він схвалив зміни годину тому.

Alex Tutor

Рев'ю на #134 на цей момент

Oscar
Approved these changes: код у порядку

Ви

Тобто моє рев'ю відповідає на питання «чи це правильна зміна для клієнтів?», а не «чи це добрий код?»

Alex Tutor

Саме так. А якщо щось у коді здається тобі дивним, постав питання. Не давай вказівок щодо того, чого не можеш оцінити.

Alex Tutor

Питання 1

Що з цього входить у рев'ю #134 від Sofia? Познач усе, що підходить.

Оберіть усі правильні відповідіЧи робить виправлення те, що описано в issue #131Чи зрозуміла назва функції в компонентіФормулювання підказки, яку побачать клієнтиЧи не буде інша бібліотека швидшоюЩо буде, коли клієнт спорожнить кошикЗміни, про які issue не просило

Alex Tutor

Почни там, де почнуть клієнти: з preview. Спершу пройди кроки з «How to test» від Daniel, а потім спробуй те, чого він не записав.

Ви

Кроки з першого по третій на телефоні працюють. Змінюю кількість, кнопка лишається зеленою. Прибираю все, вона сіріє, і з'являється підказка: «Cart is empty. Add items to checkout.»

Alex Tutor

Тепер граничні випадки (edge cases): шляхи поза щасливим сценарієм. Що ще клієнт може зробити з кошиком?

Ви

Прибирати товари по одному кнопкою «мінус», а не смітничком. Застосувати промокод, а потім змінити кількість. Відкрити кошик у двох вкладках і редагувати в обох.

Daniel

Дві вкладки це підступно. Мені подобається.

Ви

Усе працює. Бентежить лише підказка: «Add items to checkout» читається як повідомлення про помилку.

Alex Tutor

Питання 2

Що з цього є граничним випадком, який варто спробувати на preview #134, окрім кроків із «How to test» від Daniel?

Оберіть одну відповідьПеревірити, що логотип Котокорму досі завантажується на головнійЗастосувати промокод, а потім змінити кількість у кошикуПрочитати новий файл тестів рядок за рядкомПеремкнути мову сайту на сторінці доставки

Alex Tutor

Тепер Files changed. Читати кожен файл не обов'язково. Станом на 2026 рік у кожного файлу є позначка Viewed («переглянуто»): відміть ті, що вже переглянуто або лишено іншим, і вони згорнуться.

Ви

Компонент і тест це до Oscar. Лишаються copy/cart.json і styles/checkout.css.

Alex Tutor

Наведи курсор на рядок у diff, і з'явиться синій +. Натисни його або протягни через кілька рядків, і просто під рядком відкриється поле коментаря.

У поля дві кнопки. Add single comment («додати окремий коментар») публікує одразу. Start a review («почати рев'ю») зберігає коментар як чернетку, тож можна додати ще й надіслати все разом із вердиктом.

Daniel

Будь ласка, збирай усе разом. П'ять окремих коментарів це п'ять сповіщень посеред моєї роботи. Одне рев'ю це одне сповіщення з усією твоєю думкою.

Alex Tutor

Доки ти не надішлеш рев'ю, коментарі позначені Pending («очікує»), і бачиш їх лише ти.

Alex Tutor

Питання 3

Sofia починає рев'ю й залишає три коментарі з позначкою Pending. Daniel уже може їх прочитати.

ПравдаНеправда

Alex Tutor

`copy/cart.json` у #134

   "cartTitle": "Your cart",
+  "checkoutDisabledHint": "Cart is empty. Add items to checkout.",
   "promoCodeLabel": "Promo code"

Один доданий рядок: підказка під сірою кнопкою.

Ви

До того ж «checkout» це іменник, а «check out» дієслово. Тож тут не лише холодно, а й неправильно.

Alex Tutor

Тоді не описуй виправлення, а напиши його. На панелі коментаря є кнопка для запропонованої зміни (suggestion; станом на 2026 рік це маленька іконка з плюсом і мінусом). Вона вставляє блок suggestion із поточним рядком, готовим до редагування.

Alex Tutor

Коментар Sofia до цього рядка

```suggestion
  "checkoutDisabledHint": "Your cart is empty. Add a treat to check out.",
```
"Check out" is the verb here. A friendlier tone, too.

У блоці suggestion рядок, яким Sofia пропонує замінити доданий. Нижче пояснення, чому.

Daniel

У мене це виглядає як маленький diff із кнопкою Commit suggestion («закомітити пропозицію»). Один клік, і твій рядок стає комітом у моїй гілці, а GitHub додає тебе до співавторів коміту.

Alex Tutor

Пропозиції залиш для дрібних точних правок: текст, число, посилання. Для чогось більшого опиши проблему, а рішення нехай обере автор.

Alex Tutor

Питання 4

Розстав кроки, якими Sofia пропонує нове формулювання, у правильному порядку.

Торкніться пункту, а потім місця, куди його поставити. Можна перетягувати або користуватися стрілками.

Alex Tutor

`styles/checkout.css` у #134

 .checkout-button {
-  background: #2e9e5b;
+  background: #f28c28;
   border-radius: 8px;

#2e9e5b це теперішній зелений; #f28c28 помаранчевий.

Ви

Із зеленого на помаранчевий. Issue #131 нічого не каже про колір, і опис теж.

Daniel

Знову винен. Я однаково був у цьому файлі, а помаранчевий помітніший. Визнай.

Alex Tutor

Може, й так. Але це головна кнопка всього магазину, змінена для кожного клієнта, всередині виправлення бага, яке ніхто не читатиме як редизайн. Якщо наступного тижня замовлень поменшає, ніхто не знатиме, де шукати.

Ви

І вона заслуговує на нормальну перевірку: помаранчевий проти зеленого, A/B-тестом, з окремим issue.

Alex Tutor

Це й є перевірка меж зміни (scope): порівняй diff з issue й описом. Усе зайве отримує питання, а не мовчазне схвалення.

Alex Tutor

Питання 5

Чому зміна кольору не має потрапити на сайт у складі #134?

Оберіть одну відповідьВона змінює те, що бачить кожен клієнт, не входить ні в #131, ні в опис і заслуговує на окреме рішення й перевіркуGit не вміє зливати файли стилів разом із файлами кодуЛише дизайнерам дозволено змінювати стилі на GitHubПомаранчеві кнопки не проходять перевірки доступності GitHub

Alex Tutor

Тепер напиши коментар до цього рядка. Пам'ятай, хто його читатиме: колега, який витратив на це три дні й читатиме твоє рев'ю о шостій вечора.

Допомагає кілька звичок. Коментуй зміну, а не людину. Пиши, чому, а не лише що. Питай, коли маєш сумнів. Позначай дрібниці як необов'язкові. І кажи, що вдалося.

Daniel

«Навіщо ти це зробив?» і «Яка причина зміни кольору?» питають те саме. Тільки одне з них звучить як допит.

Ви

Тоді так: «Зміна кольору не входить у #131, а вона стосується кожного клієнта. Може, винесемо її в окреме issue й перевіримо там?»

Alex Tutor

Саме так. Конкретно, з причиною і з пропозицією, що робити далі.

Alex Tutor

Питання 6

Які з цих коментарів у рев'ю відповідають звичкам вище? Познач усе, що підходить.

Оберіть усі правильні відповіді«Нащо було робити її помаранчевою??»«Необов'язково: "Your cart is empty" може звучати привітніше, ніж "Cart is empty".»«Це неправильно.»«Підказка з'являється на preview, коли кошик порожній. Саме те, чого бракувало в #131, класно.»«Що буде, якщо промокод закінчиться, поки кошик відкритий? На preview я не можу це перевірити.»

Alex Tutor

Твої коментарі чекають. Щоб їх надіслати, натисни кнопку рев'ю вгорі Files changed: станом на 2026 рік вона називається Submit review («надіслати рев'ю»; у старішому інтерфейсі Review changes). Вона відкриває поле для підсумку й три варіанти.

Comment («коментар»): відгук без вердикту. Approve («схвалити»): це можна зливати. Request changes («запит на зміни»): щось має змінитися до мерджу.

Daniel

Ще до схвалень пишуть «LGTM»: «looks good to me», «як на мене, усе добре». Нормально, якщо це правда. Найгірше рев'ю це LGTM на pull request, який ніхто не відкривав.

Ви

А Start a review проти Add single comment? Де тут це?

Alex Tutor

Це попередній крок. Ці дві кнопки вирішують, чи коментар чекає на твій вердикт, чи йде окремо. А сам вердикт завжди один із цих трьох.

Alex Tutor

Питання 7

Зістав кожну ситуацію з кнопкою чи варіантом, який варто обрати Sofia.

Торкніться відповіді, а потім рядка, до якого вона належить. Можна й перетягувати.

Буде кілька коментарів до рядків, і їх треба надіслати разом
Preview працює, тексти правильні, у diff немає нічого зайвого
Виправлення працює, але pull request заодно змінює ціну доставки
Є лише питання, а вердикту поки немає

Відповіді, які ще треба розмістити

Ви

Текст це дрібна правка, а от колір для мене справжній блокер. Тож Request changes?

Alex Tutor

Так, якщо в підсумку чітко сказано, що саме блокує і чому. Не проси змін через кому і не схвалюй із надією, що колір помітить хтось інший.

Daniel

Request changes це не грубість. Це точність. Краще знати, що блокує мердж, ніж угадувати з купи коментарів.

Alex Tutor

Напиши підсумок зараз. Daniel прочитає його першим, над твоїми коментарями до рядків.

Alex Tutor

Питання 8Коротка відповідь · Завдання з перевіркою ШІ

Напиши підсумок до рев'ю Sofia з вердиктом Request changes для #134 (2–4 речення). Її коментарі до рядків: пропозиція для тексту підказки в copy/cart.json і зауваження, що зміна кольору в styles/checkout.css не входить у #131. Preview спрацював у кожному випадку, який вона пробувала.

Напишіть відповідь і отримайте оцінку та відгук від нашого ШІ-рецензента.

Увійдіть, щоб отримати відгук ШІ

Alex Tutor

Ти натискаєш Submit review. Блок мерджу внизу Conversation тепер каже «Changes requested», і мердж заблоковано, хоча Oscar схвалив зміни.

Alex Tutor

Рев'ю на #134 після того, як Sofia надсилає своє

Oscar
Approved these changes: схвалив зміни
Sofia
Requested changes: зміна кольору поза межами #131

Ви

Стоп, моє рев'ю його блокує? Daniel казав, що одного схвалення досить.

Daniel

Одне схвалення обов'язкове, а запит на зміни від будь-кого з правом запису тримає мердж, доки ця людина не схвалить або рев'ю не відхилять (dismiss). Це наш захист гілки (branch protection) для main.

Alex Tutor

Захист гілки (або ruleset у новіших налаштуваннях) це перелік правил, які GitHub примусово застосовує до гілки. Для main у Котокорму: зміни лише через pull requests, щонайменше одне схвалення, build і test мають пройти, усі обговорення розв'язані, а гілка не відстає від main.

Daniel

Останнє правило з'явилося після моєї березневої пригоди з панеллю cookie. Завтра побачиш його в дії.

Ви

А Daniel не може просто схвалити власний pull request?

Daniel

Ні. GitHub не дає авторам схвалювати власні pull requests, а до правила зараховуються лише рев'ю від людей із правом запису.

Alex Tutor

Питання 9

Правила Котокорму для main вимагають одного схвалення, а Oscar і Sofia мають право запису. Перевірки пройдено. Чи може Daniel змерджити #134 просто зараз?

Рев'ю на #134

Oscar
Sofia

Оберіть одну відповідьТак: правилу потрібне лише одне схваленняТак, але лише через Squash and mergeНі: кожен запрошений рев'юер має схвалити, хай що він написавНі: запит на зміни від Sofia блокує мердж, доки Sofia не схвалить зміни або це рев'ю не відхилять

Alex Tutor

За годину. Daniel натиснув Commit suggestion на підказці, повернув зелений другим комітом і відповів на обидва твої коментарі.

Alex Tutor

fix/checkout-button після рев'юДва нові коміти в гілці: c2d7f90 застосовує пропозицію Sofia, f7a2b61 повертає зелену кнопку. main не змінилася.

f02a6d4Update FAQ answers
3a9d1c4Reproduce bug in a test
8be2f70Add cat treats bundle
b7e4a05Re-enable button after cart edits
6f2c8a1Raise free-delivery threshold to 1000 UAH (#133)
a7d93e4Rewrite delivery page copy (#132)main
c2d7f90Update checkout hint copy
f7a2b61Restore checkout button colourfix/checkout-button

Daniel

Повідомлення для твоєї пропозиції я написав сам: «Update checkout hint copy». GitHub показує тебе серед співавторів цього коміту. Твій перший рядок коду, можна сказати.

Ви

Його відповідь під кольором: «Слушно. Помаранчевий переніс у #135, перевірмо його як слід». І він натиснув стрілки біля мого імені, щоб знову попросити моє рев'ю.

Alex Tutor

Тепер заверши коло. Кожне обговорення має кнопку Resolve conversation («позначити обговорення розв'язаним»), а правила Котокорму блокують мердж, доки відкрите хоч одне. У нас обговорення закриває той, хто його почав, коли його все влаштовує.

Ви

Мій рядок на місці, зелений повернувся. Розв'язую обидва обговорення, і цього разу: Approve.

Alex Tutor

Ще один pull request для тебе, від початку до кінця. #154 «Add gift note to checkout» відповідає на issue #139: клієнти просили додавати записку до замовлень, які вони дарують. Oscar уже схвалив код; решта твоя.

Ви

Спершу issue, потім опис, потім preview, потім diff.

Alex Tutor

Добрий порядок. Потім напиши рев'ю так, ніби надсилаєш його насправді: коментарі до рядків, пропозиція, де доречно, і вердикт із підсумком.

Alex Tutor

Питання 10Кейс · Завдання з перевіркою ШІ

Напиши своє рев'ю pull request #154: коментарі до рядків (із пропозицією, де доречно), вердикт і підсумок.

Issue #139 «Let customers add a gift note» просить необов'язкове поле під час оформлення, до 150 символів, яке друкується на пакувальному листі й не з'являється в рахунку.

Pull request #154 «Add gift note to checkout», main ← feature/gift-note. Опис: додає необов'язкову записку до подарунка під час оформлення, яка друкується на пакувальному листі; Closes #139. Усі перевірки пройдено; Oscar схвалив код.

Files changed (фрагменти):

# copy/checkout.json
+  "giftNoteLabel": "Gift mesage (optional)",
+  "giftNotePlaceholder": "A few words for the lucky cat's human",

# components/GiftNote.tsx
+  const MAX_LENGTH = 500;

# config/delivery.json
-  "defaultMethod": "courier",
+  "defaultMethod": "pickup",

На preview: записка друкується на пакувальному листі. А ще вона з'являється в PDF-рахунку. Емодзі в записці на пакувальному листі виглядає як порожній квадратик.

Рев'ю можна писати українською; текст для сайту в пропозиції лишається англійським.

Напишіть відповідь і отримайте оцінку та відгук від нашого ШІ-рецензента.

Увійдіть, щоб отримати відгук ШІ

Наступний урок: усі іконки й кольори, які ти бачиш цього тижня, і кілька нових, прочитані з першого погляду.

Daniel

А завтра зранку я зливаю #134. Ну, щойно блок мерджу перестане вимагати від мене ще одну річ.

Alex Tutor

На цьому все. Ви відповіли на всі завдання — чудова робота.

Наступний урок: Іконки й кольори: що означає кожен стан