Megosztás a következőn keresztül:

Tudnivalók a pull requestekről

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

A pull requestek (PR-k) az Azure Repos Git-adattárában lévő kód módosítására, felülvizsgálatára és egyesítésére szolgálnak. A PRS-ek származhatnak ugyanahhoz az adattárhoz tartozó ágakból vagy az adattár elágazásaiból. A csapatok a PR-ek használatával áttekintik a kódot, és visszajelzést adnak a változásokról, mielőtt a kódot egyesítenék a főágba. A véleményezők végiglépkedhetnek a javasolt módosításokon, megjegyzéseket hagyhatnak, és szavazhatnak a kód jóváhagyására vagy elutasítására.

Ez a cikk a lekéréses kérelmekre vonatkozó irányelveket és felügyeleti szempontokat ismerteti. A lekéréses kérelmek létrehozásáról, megtekintéséről, áttekintéséről és befejezéséről az alábbi cikkekben olvashat:

Feljegyzés

Teljesítmény- és stabilitási okokból a lekéréses kérelmekhez hozzáadható felülvizsgálók számának legfeljebb 1000-nek kell lennie. A rendszer nem hoz létre új lekéréses kérelmeket, ha több mint 1000 véleményezőt ad hozzá, és a meglévő lekéréses kérelmek nem teszik lehetővé több mint 1000 véleményező hozzáadását.

Jótanács

Az AI használatával segíthet az Azure DevOps-feladatokban. Az első lépésekhez tekintse meg az Azure DevOps MCP Serverrel kapcsolatos AI-segítség engedélyezése című témakört.

Engedélyek és előfeltételek

  • Repositóriumoknak engedélyezve kell lenniük a projektben. Ha a Repos és a kapcsolódó lapok nem jelennek meg, tekintse meg az Azure DevOps-szolgáltatás be- vagy kikapcsolását, hogy újra engedélyezze a Repos-t.

  • A PR-k megtekintéséhez vagy áttekintéséhez az Azure DevOps-projekt tagjának kell lennie, legalább alapszintű hozzáféréssel.

  • Ahhoz, hogy hozzájáruljon egy PR-hez, legyen a Olvasók biztonsági csoport tagja, vagy rendelkezzen a megfelelő engedélyekkel.

  • A lekéréses kérelem létrehozásához és elvégzéséhez a Közreműködők biztonsági csoport tagjának kell lennie, vagy rendelkeznie kell a megfelelő engedélyekkel.

Feljegyzés

Nyilvános projektek esetén a Stakeholder hozzáféréssel rendelkező felhasználók teljes hozzáféréssel rendelkeznek az Azure-adattárakhoz.

  • Repositóriumoknak engedélyezve kell lenniük a projektben. Ha a Repos és a kapcsolódó lapok nem jelennek meg, tekintse meg az Azure DevOps-szolgáltatás be- vagy kikapcsolását, hogy újra engedélyezze a Repos-t.
  • A PR-k megtekintéséhez vagy áttekintéséhez az Azure DevOps-projekt tagjának kell lennie, legalább alapszintű hozzáféréssel. Ha nem vagy tagja a projektnek, kérd a hozzáadásodat.
  • Ahhoz, hogy hozzájáruljon egy PR-hez, legyen a Olvasók biztonsági csoport tagja, vagy rendelkezzen a megfelelő engedélyekkel.
  • A lekéréses kérelem létrehozásához és elvégzéséhez a Közreműködők biztonsági csoport tagjának kell lennie, vagy rendelkeznie kell a megfelelő engedélyekkel.

Az engedélyekről és a hozzáférésről további információt az Alapértelmezett Git-adattár és ágengedélyek, valamint a Hozzáférési szintek ismertetése című témakörben talál.

Minőségi visszajelzés pull requestekhez

A kiváló minőségű felülvizsgálatok kiváló minőségű visszajelzésekkel kezdődnek. Íme néhány kulcs a nagy PR-visszajelzéshez:

  • A pull request tulajdonosának rendelkeznie kell a megfelelő személyekkel, hogy átnézzék a pull requestet, és biztosítania kell, hogy a véleményezők tisztában legyenek a kód funkcióival.
  • A véleményezőknek végrehajtható, konstruktív visszajelzést kell adniuk.
  • A tulajdonosoknak és a véleményezőknek gyorsan kell megjegyzéseket fűzni és válaszolniuk.

A PR-tulajdonosoknak a következőket kell tenniük:

  • Győződjön meg arról, hogy a megfelelő véleményezőket választja ki a pull requesthez való hozzárendeléshez.
  • Olyan véleményezőket is felvehet, amelyek tudják, hogyan működik a kód.
  • Kérje meg a más területeken dolgozó fejlesztőket, hogy oszthassák meg ötleteiket.
  • Adja meg a módosítások egyértelmű leírását.
  • Útmutatást adhat a véleményezőknek a lekéréses kérelmek sablonjaihoz.
  • Adja meg a kód buildjét a benne futó javítással vagy szolgáltatással.
  • Válaszoljon a megjegyzésekre, fogadja el a javaslatot, vagy magyarázza el, hogy miért nem ideális a javasolt módosítás.
  • A kérelemkérés hatókörén kívül eső jó javaslatokhoz hozzon létre új munkaelemeket, ágakat és PRS-eket a módosítások elvégzéséhez.

A véleményezőknek a következő feladatokat kell elvégeznie.

  • Visszajelzés küldése azokkal a módosításokkal kapcsolatban, amellyel nem értenek egyet
  • Problémák azonosítása és konkrét javaslatok küldése a különböző teendőkre
  • Győződjön meg arról, hogy a visszajelzés egyértelmű szándékkal rendelkezik, és könnyen érthető
  • Megjegyzések hagyása vagy szavazás a módosításokról

További információért lásd a Git lekéréses kérelmekkel visszajelzést kapni dokumentumot.

Ágszabályzatok és összevonási kérelmek

Előfordulhat, hogy a csapat az adattárban lévő kritikus ágakra, például az main ágra támaszkodik, hogy mindig jó állapotban legyen. Beállíthatók ágpolitikák úgy, hogy PR-eket követeljenek meg a védett ágakon végzett módosításokhoz, és elutasítsák a közvetlenül az ágakba leküldött változtatásokat.

További szabályzatokat is hozzáadhat a PRS-ekhez, hogy jobb kódminőséget kényszerítsen ki a kulcságakban. Az olyan további követelmények, mint a javasolt kód tiszta felépítése vagy több véleményező jóváhagyása segíthet a kulcságak védelmében.

A PR-ekhez szükséges jóváhagyások számát egy ágpolitikában állíthatja be. Beállíthatja azt is, hogy bizonyos véleményezők kötelezőek legyenek, vagy nem kötelezőek legyenek az összes vagy bizonyos PRS-en. A lekéréses kérelem beállítható automatikus kiegészítésre a szükséges számú jóváhagyással, még akkor is, ha más véleményezők elutasítják a módosításokat. Azonban a szükséges felülvizsgálóknak jóvá kell hagyniuk a PR-kat, mielőtt azok összevonódhatnak. Ajánlott, hogy legalább két felülvizsgáló tekintse át és hagyja jóvá a jelentős PR-ek változásait.

Ha vissza szeretné állítani a szavazatokat, amikor egy PR-szerző új módosításokat küld be, válassza az Új változások esetén a kódvizsgálók szavazatainak visszaállítása lehetőséget a Véleményezők minimális számának megkövetelése fiókszabályzatban.

Az alábbi táblázat az ágak testreszabásához definiálható szabályzatokat foglalja össze. Az adattár- és ágházirendek és -beállítások áttekintéséért tekintse meg a Git-adattár beállításait és szabályzatait.

Szabályzat

Alapértelmezett

Leírás

A véleményezők minimális számának megkövetelése

Kikapcsolva

A lekéréses kérelmek adott számú véleményezőjének jóváhagyására van szükség.

Csatolt munkaelemek keresése

Kikapcsolva

A nyomon követhetőség ösztönzése a csatolt munkaelemek lekéréses kérelmeken való keresésével

Ellenőrizze a megjegyzésmegoldást

Kikapcsolva

Ellenőrizze, hogy az összes megjegyzés megoldódtak-e a pull requestekben.

Egyesítési típusok korlátozása

Kikapcsolva

Az ágelőzmények szabályozása érdekében korlátozza a pull kérések teljesítésekor engedélyezett egyesítési típusokat.

Build érvényesítés irányelveinek hozzáadása

Kikapcsolva

Adjon hozzá egy vagy több szabályzatot a kód ellenőrzéséhez a lekéréses kérelmek módosításainak előzetes egyesítésével és létrehozásával. Emellett engedélyezheti vagy letilthatja a szabályzatokat.

Állapot-ellenőrzési szabályzatok hozzáadása

Kikapcsolva

Adjon hozzá egy vagy több szabályzatot, amelyek megkövetelik, hogy más szolgáltatások sikeres állapotot tegyenek közzé a lekéréses kérelmek teljesítéséhez. Emellett engedélyezheti vagy letilthatja a szabályzatokat.

Automatikusan belefoglalt véleményezők

Kikapcsolva

Adjon hozzá egy vagy több szabályt a kód véleményezőinek automatikus kijelöléséhez, amikor a lekérési kérelmek bizonyos kódterületeket módosítanak. Emellett engedélyezheti vagy letilthatja a szabályzatokat.

További információk:

Állapotellenőrzések definiálása a kódminőség javítása érdekében

A lekéréses kérelmek és az ágszabályzatok segítségével a csapatok betartatják a kód áttekintésére és az automatizált buildek futtatására vonatkozó ajánlott eljárásokat. Sok csapatnak további követelményei és ellenőrzései vannak a kóddal kapcsolatban. Ezeknek az igényeknek a kielégítése érdekében integrálhatja a PR-állapotellenőrzéseket a PR-munkafolyamatba. A PR állapot-ellenőrzésekkel a külső szolgáltatások programozott módon jóváhagyhatják a kódváltozásokat a sikeresség vagy sikertelenség információinak társításával a PR-rel.

További információért tekintse át az alábbi cikkeket:

Több egyesítési alappal kapcsolatos probléma

Bizonyos esetekben egy PR-nak több valódi egyesítési alapja van, és ez a helyzet biztonsági problémákat okozhat. Ha a lekéréses kérelemben szereplő fájlok különböző verziójúak az egyesítési alapok között, több, egyesítési alappal kapcsolatos figyelmeztetés történik. További információk és javítási megoldások: Több egyesítési alapok.

Következő lépések

  • Last updated on