11. Pull request, review, merge (10 min)

Výstup bloku: každý otevřel pull request, dostal review a jeho změna je ve main.

Pull request (3 min)

Pull request (PR) je žádost o začlenění větve do main. Je to místo, kde se o změně diskutuje dřív, než se dostane do hlavní verze.

Obsahuje:

• porovnání změn (diff),
• popis, co a proč se mění,
• komentáře k řádkům kódu,
• stav kontrol (později automatické testy).

Dobrý popis PR: co se změnilo, proč, jak to ověřit.

Code review (3 min)

Kolega změnu prohlédne a buď schválí, nebo požádá o úpravu.

Na co se dívat:

• dává to smysl a splňuje to úkol?
• je to srozumitelné?
• nerozbije to něco jiného?
• není v tom heslo nebo soubor, který tam nepatří?

Jak psát komentáře: k věci, ne k člověku. „Navrhuji…“, „Co kdyby…“, ne „Špatně“.

Merge (2 min)

Po schválení se PR sloučí tlačítkem Merge pull request. Větev lze potom smazat. Po mergi každý:

git switch main
git pull

Praxe (2 min a dále)

1. Na GitHubu se po pushi objeví nabídka Compare & pull request. Otevřít PR do main, napsat popis.
2. Každý projde PR kolegy vedle sebe, přidá aspoň jeden komentář a schválí (Review changes → Approve).
3. Autor PR sloučí.
4. Při konfliktu v web/index.html (GitHub ho ohlásí) autor větev aktualizuje:

git switch feature/jana-kontakt
git pull origin main        # vyřeší konflikt jako v předchozím bloku
git push

Tento krok je přechod mezi cvičeními, zvládne se i po bloku, pokud je čas.

Poznámky pro lektora

• Doporučit v nastavení repozitáře ochranu větve main (Settings → Branches → Branch protection rules → Require a pull request before merging). Ukázat, ne vynucovat na školení.
• U většího týmu lze využít také CODEOWNERS a šablony PR (.github/pull_request_template.md).