Pull request od vytvorenia po zlúčenie

V tomto článku sa dozviete, čo je Pull Request (PR), prečo slúži na diskusiu o zmenách a ako vytvárať a kontrolovať PR priamo v IDE. Popritom si prejdeme tri pravidlá dobrého commitu. Je to dôležitý základ: Claude Code vie pripraviť pull request aj komentovať jeho obsah. Ak chcete tieto možnosti využívať vedome, potrebujete vedieť, čo PR znamená a na čo slúži.

Príklady a snímky rozhrania sa týkajú IDE JetBrains; názvy a umiestnenie akcií sa vo vašej verzii môžu líšiť. Na cvičenia použite samostatný učebný repozitár. V pracovnom projekte dodržiavajte jeho pravidlá pre vetvy, vytváranie commitov a odosielanie zmien.

Načo slúžia Pull Requests

V tíme sa oplatí pred zlúčením do spoločnej vetvy prediskutovať zmeny. Pull request zhromažďuje navrhovaný diff, pripomienky a výsledky kontrol na jednom mieste. Používa sa aj v osobných projektoch, keď si chcete pred zlúčením skontrolovať vlastnú prácu.

Pri práci s PR najprv vytvoríte návrh na zlúčenie. Na GitHube sa nazýva Pull Request, na niektorých iných platformách Merge Request. Konkrétny pracovný postup určuje tím.

Pull Request je oficiálna žiadosť: "prosím, stiahnite (pull) moje zmeny z mojej vetvy a začleňte (merge) ich do hlavnej vetvy". Nie je to však len žiadosť, ale priestor na diskusiu, kontrolu a vylepšovanie kódu predtým, než sa dostane do main.

Bežný pracovný postup s Pull Requestom

Pozrime sa krok za krokom, ako zvyčajne prebieha pridávanie novej funkcie.

Pred vytvorením vetvy skontrolujte východiskovú vetvu a zoznam jej commitov v porovnaní s cieľovou vetvou na GitHube. Do nového PR sa nesmú dostať nesúvisiace lokálne commity. Neodosielajte ich automaticky len kvôli "synchronizácii": vyberte správny základ úlohy a predchádzajúcu prácu si odložte bokom.

Krok 1. Vytvorte vetvu pre novú úlohu. V tomto postupe s PR sa práca začína v samostatnej vetve. Predstavme si, že chceme pridať súbor s pravidlami pre prispievateľov - vytvorme vetvu feature/add-contribution-guide.

Krok 2. Urobte zmeny a odošlite vetvu na GitHub. V novej vetve vytvorte súbor CONTRIBUTING.md (po vytvorení kliknite na Add) a napíšte doň odkaz pre ostatných vývojárov. Potom použite Commit and Push s výstižnou správou, napríklad docs: Add contribution guide.

Toto je kľúčový krok. Pull Request sa vytvára z vetvy, ktorá už existuje na vzdialenom serveri. Preto novú vetvu musíte pred vytvorením PR odoslať (push) na GitHub.

Vytvorenie Pull Requestu z IDE

Skontrolujte obe strany porovnania: base je vetva, do ktorej navrhujete zmeny, compare/head je vetva s vašou prácou. Pred vytvorením PR si prezrite zoznam commitov a diff.

Keď už je vaša vetva na GitHube, môžete vytvoriť Pull Request.

Krok 1. Otvorte kartu Pull Requests. Na ľavom bočnom paneli IDE nájdite kartu Pull Requests. Otvorte ju a kliknite na ikonu + (alebo na tlačidlo Create Pull Request).

Krok 2. Vyplňte údaje PR. IDE otvorí prehľadné rozhranie. Vašou úlohou je správne ho vyplniť. Hlavné polia:

1. nadpis - IDE sem často doplní názov vetvy alebo správu posledného commitu; skontrolujte, či je nadpis stručný, zrozumiteľný a vystihuje zmeny;
2. opis - podrobne vysvetlite kolegom, čo ste urobili a prečo;
3. recenzenti (Reviewers) - ľudia, ktorých požiadate o kontrolu zmien. Či je ich schválenie povinné, určujú pravidlá repozitára;
4. riešitelia (Assignees) - zvyčajne sem uvediete seba ako autora a osobu zodpovednú za úpravy po review.

Pred kliknutím na Create Pull Request ešte raz skontrolujte base, head a diff. Ak práca ešte nie je pripravená na review a platforma to umožňuje, môžete vytvoriť Draft PR. Bežný PR aj draft už vidia ľudia s prístupom k repozitáru.

Code review - kontrola a diskusia

Po vytvorení PR prichádza najdôležitejšia fáza - code review. Kolegovia si môžu otvoriť váš PR, prezrieť všetky zmeny a pridať komentáre.

Všetky diskusie uvidíte priamo v IDE na karte Pull Requests. Ak niekto pridá komentár k riadku kódu, dostanete upozornenie a môžete odpovedať bez prepínania do prehliadača.

Čo robiť, keď vás požiadajú o úpravy? Je to jednoduché: nový PR netreba vytvárať. Upravte kód v pôvodnej vetve tohto PR, vytvorte nový commit a odošlite ho. Pull Request na GitHube sa automaticky aktualizuje a k existujúcej diskusii pribudnú nové commity.

Dokončenie práce - zlúčenie a odstránenie vetvy

Keď vyriešite všetky pripomienky a tím zmeny schváli (dostanete "approval"), Pull Request môžete zlúčiť. Zvyčajne to urobí skúsenejší vývojár alebo vy, ak máte v repozitári príslušné oprávnenia.

Krok 1. Zlúčenie. Skontrolujte pripomienky, povinné schválenia a stav kontrol. Ak to pravidlá a vaše oprávnenia umožňujú, vyberte podľa pravidiel projektu dostupný spôsob (merge, squash alebo rebase) a potvrďte ho. Zmeny sa dostanú do cieľovej vetvy PR, v našom príklade do main.

Krok 2. Odstránenie vetvy. Po zlúčení už feature vetvu nepotrebujete, preto ju odstráňte, aby v repozitári nezostával zbytočný neporiadok. GitHub vám to ponúkne tlačidlom Delete branch.

Po zlúčení sa lokálne prepnite na main, stiahnite aktualizácie a overte, že sa v ňom nachádza výsledok PR. Potom môžete odstrániť lokálnu vetvu úlohy, ak v nej nezostala potrebná práca. Ak ju IDE nedovolí odstrániť bežným spôsobom, najprv zistite prečo, najmä po squash alebo rebase.

Tri pravidlá dobrého commitu

Pri dobrom commite záleží nielen na správe, ale aj na obsahu. Aby bola história zmien prehľadná a užitočná, držte sa troch jednoduchých pravidiel.

Pravidlo 1 - píšte zrozumiteľné správy

Správy vašich commitov sú odkazom pre tím aj pre vaše budúce ja. História plná samotných "fix" alebo "update" je zbytočná. Jedným z rozšírených formátov je Conventional Commits, ktorý používa túto štruktúru:

<typ>: <stručný opis>

Typ je krátke slovo, ktoré opisuje kategóriu zmeny:

• feat - nová funkcia;
• fix - oprava chyby;
• docs - zmeny v dokumentácii;
• style - úpravy formátovania, ktoré neovplyvňujú logiku;
• refactor - zmeny kódu bez novej funkcie a bez opravy chýb;
• test - pridanie alebo oprava testov;
• chore - rutinné úlohy (aktualizácia závislostí, nastavenie zostavenia).

Príklady:

• Nevhodné: fixed bug
• Vhodné: fix: Correct user login validation
• Nevhodné: readme
• Vhodné: docs: Update installation instructions

Pravidlo 2 - jeden commit znamená jednu logickú zmenu

Nesnažte sa vtesnať do jedného commitu opravu chyby, novú funkciu aj refactoring starého kódu. Taký commit sa ťažko kontroluje pri review a takmer sa nedá bezbolestne vrátiť. Každý commit má riešiť jednu konkrétnu úlohu.

• Nevhodné: jeden commit "Update user page", ktorý pridá pole pre avatar, opraví kontrolu mena a zmení farbu tlačidiel;
• Vhodné: tri samostatné commity:
  1. feat: Add avatar upload to user profile
  2. fix: Correct username validation logic
  3. feat: Update button colors on user page

Ak ste pri práci upravili v jednom súbore priveľa vecí, IDE umožňuje commitnúť len jeho časť. V okne Commit môžete súbor rozbaliť a zaškrtnúť iba tie úseky (chunks), ktoré patria ku konkrétnej úlohe.

Pravidlo 3 - kontrolujte výsledok zmien

Pred zlúčením spustite kontroly vhodné pre danú zmenu a pozrite si ich skutočné výsledky. Pri úprave dokumentácie a pri novej logike sa budú kontroly líšiť.

Ak sú v repozitári nastavené automatické kontroly, PR zobrazuje ich stav. Zelená značka znamená, že konkrétne spustené kontroly prešli, nie je to však dôkaz, že v kóde nie sú chyby. Pozrite si, čo sa spustilo a čo zostalo neoverené. Nastavenie automatizácie je samostatná téma.

Ako to funguje s Claude Code

Práve pri PR je Claude Code obzvlášť užitočný. Na čo sa zamerať:

• Claude dokáže vytvoriť pull request z vašej vetvy s pripraveným nadpisom a opisom - pred vytvorením PR na GitHube skontrolujete text;
• môže skontrolovať zmeny v PR a pridať komentáre ako prvý pár očí;
• pomáha vytvárať správy commitov podľa pravidiel projektu;
• môže pomôcť interpretovať výsledky už nastavených kontrol a označiť neoverené časti.

Predchádzajúci článok: Vetvy a zlúčenie v Gite · Nasledujúci článok: Vrátenie a obnova zmien v Gite