Spolupráce na pull requestech
Žádosti o přijetí změn umožňují ostatním informovat o změnách, které jste nasdíleli do úložiště GitHub.
Po odeslání pull requestu mohou zúčastněné strany zkontrolovat sadu změn, prodiskutovat možné úpravy a v případě potřeby dokonce odeslat následné commity.
Pull requesty běžně používají týmy a organizace, které spolupracují pomocí modelu sdíleného repozitáře.
Všichni sdílejí jedno úložiště a větve témat se používají k vývoji funkcí a izolaci změn.
Mnoho open-source projektů na GitHubu používá pull requesty ke správě změn od přispěvatelů.
Pomáhají poskytnout způsob, jak upozornit správce projektu na provedené změny.
Před sloučením do hlavní větve také spusťte kontrolu kódu a obecnou diskuzi o sadě změn.
Žádosti o přijetí změn kombinují kontrolu a sloučení kódu do jednoho procesu pro spolupráci.
Jakmile dokončíte opravu chyby nebo přidání nové funkce ve větvi, vytvořte novou pull request.
Přidejte členy týmu do žádosti o přijetí změn, aby mohli zkontrolovat a hlasovat o vašich změnách.
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í nutné sloučit změny, protože vlastník může žádost o přijetí změn kdykoli opustit.
Kontrola kódu
Kontrola kódu provedená v pull requestu není jen hledání chyb – to je úkolem vašich testů.
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 váš tým před nesprávnými sloučeními a poškozenými buildy, které snižují produktivitu vašeho týmu.
Kontrola tyto problémy před sloučením zachytí a chrání důležité větve před nežádoucími změnami.
Šiřte odborné znalosti a strategie řešení problémů tím, že ve svých kontrolách kódu zapojíte širokou škálu recenzentů.
Šíření dovedností a znalostí činí váš tým robustně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ří:
- Zajistěte, aby správní lidé pull request zkontrolovali.
- Ujistěte se, že revidoři vědí, co kód dělá.
- Poskytněte akční a konstruktivní zpětnou vazbu.
- Odpovězte na komentáře okamžitě.
Při přiřazování revidujících k žádosti o přijetí změn se ujistěte, že jste vybrali správnou sadu revidujících.
Chcete kontrolory, kteří vědí, jak váš kód funguje, a také zahrnout vývojáře pracující v jiných oblastech, aby sdíleli své nápady.
Navíc, kdo může poskytnout jasný popis vašich změn a sestavit váš kód tak, aby v něm běžela vaše oprava nebo funkce.
Revidujícím by se měli pokusit poskytnout zpětnou vazbu ohledně změn, se kterými nesouhlasí. Identifikujte problém a dejte konkrétní návrh na to, co byste udělali 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 žádosti o přijetí změn by měl odpovědět na komentáře, přijmout návrh nebo vysvětlit, proč navrhovaná změna není ideální.
Někdy je návrh dobrý, ale změny jsou 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.
Ochrana větví pomocí zásad
Vaše repozitáře obvykle obsahují jednu nebo více větví, včetně hlavní, které kvůli jejich důležitosti vyžadují zvýšenou ochranu. Azure Repos nabízí několik mechanismů založených na zásadách, které byste měli zvážit zavést, abyste dosáhli tohoto cíle.
Základním předpokladem těchto mechanismů je použití omezení na pull requesty. Můžete například zahrnout vynucení konkrétních zásad revize kódu, například vyžadování minimálního počtu schválení od určených recenzentů před sloučením pull requestu. Využití kolektivních odborných znalostí je vázáno na zvýšení kvality a spolehlivosti změn kódu.
Kromě toho zvažte implementaci zásady Kontrola propojených pracovních položek. To ověřuje, že každý pull request je propojen s pracovní položkou, poskytuje kontext a podporuje sledovatelnost. Zásada pro kontrolu řešení komentářů pomáhá zajistit, že před sloučením pull requestu jsou vyřešeny všechny komentáře ke kontrole kódu.
Zásady související s automatizovanou analýzou kódu, testováním a kontrolou shody potvrzují, že změny splňují předem stanovené standardy před integrací. Omezení typů sloučení pomáhá udržovat historii větví řízení. Máte například možnost povolit pouze rychlé a squashové sloučení.
Je také možné požadovat povinné čisté sestavení nových verzí kódu před povolením jejich sloučení do kritických větví. Tím zajistíte, že sloučené změny nezavádějí chyby sestavení nebo problémy s regresí. Kontroly stavu lze použít k tomu, aby dokončení žádostí o přijetí změn bylo podmíněno signály generovanými externími službami. Tyto signály může například generovat služba Azure Pipelines, která spouští automatizované testy a analýzu kódu.
Všechny větve, které mají nakonfigurované požadované zásady, automaticky blokují přímé odesílání a efektivně vyžadují žádosti o přijetí změn pro všechny úpravy. Kromě toho tyto větve nelze odstranit.