Pull request a kontrola změn

Pull request je návrh: „Tady je práce z mojí větve. Podívej se na ni, a jestli je v pořádku, přidej ji do hlavní linie.“

Vzniklo to kvůli týmům. Ale funguje to i pro jednoho člověka.

Co v něm je

1. Název a popis

   Co změna dělá a proč. Píše to autor – tedy ty, nebo Claude Code na tvůj pokyn.

2. Seznam commitů

   Historie práce ve větvi.

3. Souhrnný diff

   Všechny změny na jednom místě, soubor po souboru. Tohle je ta hodnotná část.

4. Automatické kontroly

   Když jsou nastavené, běží tu build a testy. Zeleně nebo červeně, ještě před sloučením.

Proč to dělat, i když jsi sám

Protože souhrnný diff je nejlepší kontrolní bod, jaký existuje.

Během práce vidíš změny po jedné a každá dává smysl. Pull request ti ukáže všechny najednou – a tam se objevují věci, které jednotlivě neviděl nikdo:

• Soubor, který se změnil dvakrát a podruhé se vrátil skoro do původního stavu.
• Testovací kód, který tam zůstal.
• Změna, o kterou jsi neprosil a proklouzla mezi ostatními.
• Nová závislost, kterou jsi odsouhlasil ve spěchu.

Jak ho vytvořit

Práce v téhle větvi je hotová. Vytvoř pull request na GitHubu.

Do popisu napiš:
1. Co změna dělá – z pohledu uživatele, ne z pohledu kódu.
2. Proč to bylo potřeba.
3. Co jsem měl otestovat ručně, než to sloučím.
4. Jestli je něco, čím si nejsi jistý.

Česky. Pak mi pošli odkaz.

Bod 4 je nejcennější. Nechává prostor na to, aby ti agent řekl, kde si nebyl jistý – a to je přesně místo, kam se máš podívat.

Co v diffu hledat

Platí pět otázek z lekce o čtení změn, plus dvě navíc pro pull request:

1. Sedí rozsah na název?

   Když se pull request jmenuje „Oprava překlepu v patičce“ a mění třicet souborů, něco nesedí.

2. Prošly automatické kontroly?

   Červený křížek znamená, že build neprojde. Nikdy neslučuj přes červenou s tím, že „to spravím potom“.

Projdi tenhle pull request, jako bys ho dostal ke kontrole od někoho jiného,
a buď kritický.

Hledej:
1. Změny, které nesouvisí s názvem a popisem.
2. Kód, který tam zbyl omylem – testovací výpisy, zakomentované bloky.
3. Místa, kde se něco může chovat jinak, než popis slibuje.
4. Cokoliv, co vypadá jako tajemství nebo osobní data.
5. Věci, které se rozbijí jinde v projektu.

Ke každému nálezu napiš, jak je vážný.