👨‍💻Про пулреквести в команді👨‍💻

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

А от коли ти в команді, то ПР стає дуже важливою річчю. По-перше це спосіб розділити знання з командою. Ви показуєте що ви зробили і як. По-друге, це спосіб запобігти якихось прикрих помилок, про які ви або не знаєте, або просто не бачите через звичку. По-третє, якщо рев'ювер хороший, він може підказати й більш загальні речі, наприклад що замість вашого велосипеду на 431 рядок є функція от прямо в самій лібі, і таке бувало.

Але для того щоб ваші ПР-и проходили швидко і з мінімум тертя є кілька правил яких я раджу дотримуватися.

👉 Один ПР - одна задача. Тобто один баг, одна таска, одна сторя. Рефакторинг, також окремим PR, як би не хотілося.
👉 З першого правила витікає друге правило - мінімальність. Уявіть собі що цей PR приходить до вас, а вам ще треба дві таски до вечора зробити. Який би ПР ви хотіли бачити? Звісно маленький та акуратний, і щоб нічого зайвого.
👉 Smoke Test перед PR-ом обов'язковий. Тобто підготували ПР, запустили код, перевірили що все працює. Нічого так не дратує рев'ювера як не працюючий код в PR.
👉 Корисний опис PR. Це лінк на таску, по-пунктах що зроблено (високорівнево), можна ще й скрін нового UI докласти (наприклад як ваш новий UI виглядає на маленькому екрані)

Для PR в Open-Source опис буде трохи іншим (бо там немає тасок, в більшості), але всі інші правила залишаються такими самими.

Вдалих вам ПР-ів, і скоро побачимось!
@reactbeginners