Informace o pull requestech

Azure DevOps Services | Azure DevOps Server | Azure DevOps Server 2022

Žádosti o přijetí změn představují způsob, jak změnit, zkontrolovat a sloučit kód v úložišti Git v Azure Repos. Žádosti o přijetí změn můžou pocházet z větví ve stejném úložišti nebo z větví ve forcích úložiště. Týmy používají žádosti o přijetí změn (Pull Requests) ke kontrole kódu a dávání zpětné vazby ke změnám před sloučením kódu do hlavní větve. Revidující mohou projednávat navrhované změny, zanechat komentáře a hlasovat pro schválení nebo odmítnutí kódu.

Tento článek popisuje zásady pro žádosti o přijetí změn a úvahy o řízení. Pokyny k vytvoření, zobrazení, kontrole a dokončení žádostí o přijetí změn najdete v následujících článcích:

Poznámka:

Z důvodů výkonu a stability musí být počet revidujících, kteří je možné přidat do žádosti o přijetí změn, 1 000 nebo méně. Nové žádosti o přijetí změn nebudou vytvořeny při přidávání více než 1 000 revidujících a stávající žádosti o přijetí změn vám nedovolí přidat více než 1 000 revidujících.

Návod

Pomocí AI můžete pomoct s úlohami Azure DevOps. Informace o tom, jak začít, najdete v tématu Povolení pomoci s AI s Azure DevOps MCP Serverem .

Oprávnění a požadavky

  • Úložiště musí být ve vašem projektu povolené. Pokud se centrum Repos a přidružené stránky nejsou viditelné, podívejte se na Zapnutí nebo vypnutí služby Azure DevOps pro opětovné povolení Repos.

  • Pokud chcete zobrazit nebo zkontrolovat PR, musíte být členem projektu Azure DevOps s alespoň Základním přístupem.

  • Pokud chcete přispět ke změnám, musíte být členem bezpečnostní skupiny Čtenáři nebo mít odpovídající oprávnění.

  • Chcete-li vytvořit a dokončit žádost o přijetí změn, musíte být členem skupiny zabezpečení přispěvatelů nebo mít příslušná oprávnění.

Poznámka:

U veřejných projektů mají uživatelé udělený přístup účastníka k Azure Repos plný přístup.

  • Úložiště musí být ve vašem projektu povolené. Pokud se centrum Repos a přidružené stránky nejsou viditelné, podívejte se na Zapnutí nebo vypnutí služby Azure DevOps pro opětovné povolení Repos.
  • Pokud chcete zobrazit nebo zkontrolovat PR, musíte být členem projektu Azure DevOps s alespoň Základním přístupem. Pokud nejste členem projektu, budete přidáni.
  • Pokud chcete přispět ke změnám, musíte být členem bezpečnostní skupiny Čtenáři nebo mít odpovídající oprávnění.
  • Chcete-li vytvořit a dokončit žádost o přijetí změn, musíte být členem skupiny zabezpečení přispěvatelů nebo mít příslušná oprávnění.

Další informace o oprávněních a přístupu najdete v sekcích Výchozí oprávnění pro úložiště Git a větve a Informace o úrovních přístupu.

Kvalitní zpětná vazba pro pull requesty

Vysoce kvalitní recenze začínají vysoce kvalitními názory. Tady je několik tipů pro skvělou zpětnou vazbu v oblasti public relations:

  • Vlastník žádosti o přijetí změn by měl zajistit, aby žádost kontrolovali ti správní lidé, a ujistit se, že recenzenti rozumějí tomu, co kód dělá.
  • Revidující by měli poskytnout užitečná a konstruktivní zpětnou vazbu.
  • Vlastníci a recenzenti by měli rychle komentovat a odpovídat.

Vlastníci PR by měli:

  • Ujistěte se, že vyberete správné recenzenty, kterým chcete přiřadit pull request.
  • Zahrňte recenzenty, kteří vědí, jak kód funguje.
  • Požádejte vývojáře, kteří pracují v jiných oblastech, aby sdíleli své nápady.
  • Zadejte jasný popis změn.
  • Poskytněte pokyny pro revidující pomocí šablon pull requestů.
  • Poskytněte sestavení kódu se spuštěnou opravou nebo funkcí.
  • Odpovězte na komentáře, přijměte návrh nebo vysvětlete, proč navrhovaná změna není ideální.
  • Pro vhodné návrhy mimo rozsah žádosti o přijetí změn vytvořte nové pracovní položky, větve a žádosti o přijetí změn, abyste tyto změny provedli.

Revidoři by měli provádět následující úlohy.

  • Poskytnutí zpětné vazby ke změnám, se kterými nesouhlasí
  • Identifikace problémů a poskytnutí konkrétních návrhů na to, co dělat jinak
  • Ujistěte se, že zpětná vazba má jasný záměr a je snadno pochopitelná.
  • Zanechat komentáře nebo hlasovat o změnách

Další informace najdete v tématu Získání zpětné vazby s žádostmi o úpravy v Git.

Zásady větví a pull requesty

Váš tým může spoléhat na kritické větve ve vašem úložišti, například na větev main, aby byly vždy v dobrém stavu. Můžete nastavit zásady větví tak, aby vyžadovaly žádosti o přijetí změn pro všechny změny v těchto chráněných větvích a odmítaly jakékoliv změny vložené přímo do těchto větví.

Do pull requestů můžete přidat další zásady, které zajišťují lepší kvalitu kódu v klíčových větvích. Další požadavky, jako je čistý build navrhovaného kódu nebo schválení od více recenzentů, mohou pomoci chránit klíčové větve.

V zásadách větve můžete nastavit počet požadovaných schválení pro PR. U všech nebo některých žádostí o přijetí změn můžete také nastavit, kteří recenzenti budou povinní nebo volitelní. Žádost o přijetí změn se dá nastavit tak, aby se automaticky dokončuje požadovaným počtem schválení, a to i v případě, že změny odmítli jiní revidujícím. Povinní kontroloři musí schválit PR, než může být PR sloučen. Osvědčeným postupem je, aby alespoň dva recenzenti zkontrolovali a schválili změny ve významném pull requestu.

Pokud chcete resetovat hlasy pokaždé, když autor žádosti o přijetí změn odešle nové změny, vyberte Resetovat hlasy revidujících při nových změnách v zásadách větve Vyžadovat minimální počet revidujících.

Následující tabulka shrnuje zásady, které můžete definovat pro přizpůsobení větve. Přehled všech zásad a nastavení úložiště a větví najdete v tématu Nastavení a zásady úložiště Git.

Zásady

Výchozí

Popis


Vypnuto

Vyžaduje schválení od určeného počtu recenzentů na žádostech o schválení změn.

Vypnuto

Podporujte sledovatelnost zkontrolováním propojených pracovních položek v žádostech o změny

Vypnuto

Zkontrolujte, jestli byly všechny komentáře vyřešeny u pull requestů.

Vypnuto

Řiď(te) historii větví omezením dostupných typů sloučení při dokončení pull requestů.

Vypnuto

Přidejte jednu nebo více zásad pro ověření kódu tak, že sloučíte předem a sestavíte změny pull requestů. Může také povolit nebo zakázat zásady.

Vypnuto

Přidejte jednu nebo více zásad, které vyžadují, aby ostatní služby zveřejnily úspěšný stav pro dokončení pull requestů. Může také povolit nebo zakázat zásady.

Vypnuto

Přidejte jednu nebo více zásad pro specifikaci recenzentů kódu, kteří budou automaticky zahrnuti, když žádosti o přijetí změn změní určité oblasti kódu. Může také povolit nebo zakázat zásady.

Další informace naleznete v tématu:

Definování kontrol stavu pro zlepšení kvality kódu

Žádosti o přijetí změn a zásady větví umožňují týmům vynucovat osvědčené postupy pro kontrolu kódu a spouštění automatizovaných sestavení. Mnoho týmů má k dispozici další požadavky a ověřování kódu. Pokud chcete tyto potřeby pokrýt, můžete do pracovního postupu žádosti o přijetí změn integrovat kontroly stavu žádosti o přijetí změn. Při kontrolách stavu pull requestu mohou externí služby programově schválit změny kódu tím, že přidruží informace o úspěchu nebo selhání k pull requestu.

Další informace najdete v následujících článcích:

Problém s více základy slučování

V některých případech má PR více než jednu skutečnou základnu slučování, a tato situace může způsobit bezpečnostní problémy. Pokud mají soubory v pull requestu rozdílné verze mezi základnami sloučení, zobrazí se upozornění na více základen sloučení. Pro více informací a řešení problému viz Vícenásobné základny sloučení.

Další kroky