Získejte zpětnou vazbu prostřednictvím pull requestů.

Žádosti o přijetí změn podporují kontrolu a sloučení kódu do jednoho procesu pro spolupráci. Jakmile vývojář přidá funkci nebo opravu chyb, vytvoří žádost o přijetí změn, aby zahájil proces sloučení změn do upstreamové větve. Ostatní členové týmu pak mají možnost kód před dokončením zkontrolovat a schválit. Pomocí žádostí o přijetí změn můžete zkontrolovat probíhající práci a získat včasnou zpětnou vazbu ke změnám. Není ale nutné sloučit změny. Vlastník může pull request kdykoli opustit.

Zkontrolovat kód

Kontrola kódu provedená jako součást žádosti o přijetí změn není jen nalezení jasných chyb; to jsou testy. Dobrá kontrola kódu zachytí méně běžné problémy, které by mohly vést k nákladným problémům později.

Revize kódu pomáhají chránit tým před chybným sloučením a nefunkčními buildy, které oslabují produktivitu týmu. Zkontroluje problémy před sloučením a chrání důležité větve před nežádoucími změnami.

Revize kódu také podporují a posilují spolupráci a komunikaci mezi vývojáři. A tým získá jasnou historii všech změn provedených mezi hlavní větví a funkčními větvemi.

Propojovat odborné znalosti a šířit strategie řešení problémů využitím různorodé skupiny revidujících během kontrol kódu. Šíření dovedností a znalostí činí tým silnějším a odolnějším.

Dejte skvělý názor

Vysoce kvalitní recenze začínají vysoce kvalitními názory. Mezi klíče pro skvělou zpětnou vazbu v žádosti o přijetí změn patří:

• Požádejte správné lidi, aby zkontrolovali pull request.
• Ujistěte se, že revidoři vědí, co kód dělá.
• Poskytněte akční a konstruktivní zpětnou vazbu.
• Odpovídání na komentáře včas.

Když přiřadíte recenzenty k žádosti o přijetí změn, nezapomeňte vybrat správnou skupinu recenzentů. Revidující by měli vědět, jak kód funguje, a také zahrnout vývojáře pracující v jiných oblastech, aby mohli sdílet své nápady.

Zadejte jasný popis změn a zadejte build kódu, který obsahuje opravu nebo funkci, která v ní pracuje. Revidujícím by se měli snažit poskytnout zpětnou vazbu ke změnám, se kterými nesouhlasí. Identifikujte problém a poskytněte konkrétní návrhy na to, co by se dalo udělat jinak. Tato zpětná vazba má jasný záměr a je pro vlastníka žádosti o přijetí změn snadno pochopitelná.

Vlastník pull requestu by měl odpovědět na komentáře, přijmout návrhy nebo vysvětlit, proč je odmítne použít. Některé návrhy jsou dobré, ale můžou být mimo rozsah žádosti o přijetí změn. Vezměte tyto návrhy a vytvořte nové pracovní položky a větve funkcí odděleně od žádosti o přijetí změn, aby se tyto změny udělaly.

Chraňte větve pomocí zásad

V úložišti je několik důležitých větví, na které se týmy spoléhají, že jsou vždy v dobrém stavu, jako například větev main. Týmy mohou vyžadovat návrhy na změny, aby provedly jakékoli změny na těchto větvích na platformách, jako jsou GitHub a Azure DevOps. Vývojářům, kteří odesílají změny přímo do chráněných větví, budou odmítnuty odeslané změny.

Přidejte do žádostí o přijetí změn další podmínky pro vynucení vyšší úrovně kvality kódu v klíčových větvích. Čistý build sloučeného kódu a schválení od více recenzentů jsou některé další požadavky, které se často používají k ochraně klíčových větví.

Další informace

GitHub obsahuje rozsáhlou dokumentaci k návrhu změn v práci s žádostmi o přijetí změn.

Přečtěte si další informace o poskytování skvělých připomínek v kontrolách kódu a použití šablon žádostí o přijetí změn k poskytování pokynů revidujícím. Azure DevOps také nabízí rozsáhlé prostředí pro pull requests, které je snadno použitelné a škálovatelné podle potřeby.