Oefening: een pull-aanvraag maken, beoordelen en samenvoegen
U hebt het werk voltooid om een wachtrij toe te voegen aan uw website. Het team voor websiteontwikkeling is nu klaar om de wijzigingen in uw hoofdbranch samen te voegen. In deze oefening maakt en voegt u een pull-aanvraag voor uw wijzigingen samen.
Tijdens het proces gaat u het volgende doen:
- Maak een pull-aanvraag.
- Controleer de pull-aanvraag.
- Voltooi de pull-aanvraag.
- Controleer of de wijzigingen zijn samengevoegd.
Een pull-aanvraag maken om de functiebranch samen te voegen
Omdat u wijzigingen niet rechtstreeks naar de hoofdbranch van uw opslagplaats kunt pushen, moet u een pull-aanvraag maken.
Ga in uw browser naar Code.
Selecteer 2 vertakkingen om de vertakkingen in uw GitHub-opslagplaats weer te geven.
Selecteer naast wachtrij voor bestellingenhet pictogram Meer (...), en selecteer vervolgens Nieuwe pull-aanvraag indienen.
Wanneer u de pull-aanvraag hebt gemaakt, ziet u dat GitHub automatisch het Git-doorvoerbericht heeft gebruikt als de titel van de pull-aanvraag.
Werk de beschrijving bij naar de volgende tekst:
Met deze pull-aanvraag wordt een nieuwe Azure Storage-wachtrij toegevoegd voor het verwerken van orders en wordt de websiteconfiguratie bijgewerkt om het opslagaccount en de wachtrijgegevens op te nemen.
Selecteer pull-aanvraag maken.
Ga in uw browser naar Repositories>Bestanden.
U ziet dat In Azure DevOps een banner wordt weergegeven die aangeeft dat er wijzigingen zijn in de add-orders-queue branch. De banner biedt aan om een pull-aanvraag te maken voor deze wijzigingen.
Selecteer Een pull-aanvraag maken.
Op de pagina voor het maken van een pull-aanvraag ziet u dat Azure DevOps het Git-doorvoerbericht automatisch heeft gebruikt als titel van de pull-aanvraag.
Werk de beschrijving bij naar de volgende tekst:
Met deze pull-aanvraag wordt een nieuwe Azure Storage-wachtrij toegevoegd voor het verwerken van orders en wordt de websiteconfiguratie bijgewerkt om het opslagaccount en de wachtrijgegevens op te nemen.
Selecteer maken.
De pull-aanvraag controleren
Normaal gesproken wordt een pull-aanvraag beoordeeld door iemand anders dan de auteur. In dit voorbeeld doet u zich voor als een ander teamlid en controleert u uw eigen pull-aanvraag.
Selecteer op de pagina met pull-aanvragen het tabblad Bestanden gewijzigd.
GitHub toont u de bestanden die zijn gewijzigd in deze pull-aanvraag. U ziet dat alle lijnen worden gemarkeerd die zijn gewijzigd, zodat u gemakkelijk kunt zien wat u moet controleren.
Tip
Stel dat u dit bekijkt voor uw eigen team. Zou u suggesties doen?
Beweeg in het bestand main.bicep dat is gewijzigd de muisaanwijzer over regel 18 en kies de knop met het plusteken (+).
nl-NL:
Voer in het opmerkingenvak de volgende tekst in: Moet dit hoofdlettergebruik zijn?
Selecteer Een beoordeling starten.
Tip
Met GitHub kunt u uw eigen pull-aanvragen niet goedkeuren. Hier plaatst u een commentaar op uw pull request, maar u keurt deze niet goed. Wanneer u met de pull-aanvragen van uw eigen team werkt, is dit het punt waarop u het goedkeurt om aan te geven dat u tevreden bent dat het wordt samengevoegd.
Selecteer Voltooi uw beoordeling.
Selecteer in het controlevenster dat wordt weergegeven Beoordeling verzenden.
GitHub retourneert u naar het tabblad Gesprek van de pull-aanvraag.
Selecteer op de pull request-pagina het tabblad Bestanden.
Azure DevOps toont u de bestanden die zijn gewijzigd in deze pull-aanvraag. U ziet dat alle gewijzigde regels gemarkeerd worden, zodat u eenvoudig kunt zien wat u moet beoordelen.
Fooi
Stel dat u dit bekijkt voor uw eigen team. Zou u suggesties doen?
In het bestand main.bicep dat is gewijzigd, zweef boven regel 18 en selecteer de knop Commentaar.
Voer in het opmerkingenvak de volgende tekst in: Moet dit hoofdlettergebruik zijn?
Selecteer en opmerking.
De breedte van je browservenster kan van invloed zijn op hoe het opmerkingenvenster wordt weergegeven. De opmerking opent een dialoogvenster Discussie in plaats van de inline-opmerking, zoals wordt weergegeven in de schermafbeelding.
Selecteer goedkeuren.
Nadat u Goedkeurenhebt geselecteerd, verandert de instelling Automatisch voltooien naar Volledig. U zult deze functie later in deze eenheid gebruiken.
Reageren op de beoordeling van de pull-aanvraag
Wanneer u een pull-aanvraag maakt of bekijkt, kunt u deelnemen aan een gesprek over de inhoud ervan. Stel dat u de auteur van dit bestand bent en dat u wilt reageren op een opmerking van de revisor.
Reageer op de beoordeling van de pull-aanvraag met de volgende opmerking: Nee, opslagwachtrijen moeten kleine letters hebben.
Selecteer opmerkingen selecteer vervolgens Markeer als opgelost om aan te geven dat de discussie in de conversatie voorbij is.
Selecteer op de pull request-pagina het tabblad Overzicht.
Stel dat u de auteur van dit bestand bent. Reageer op de beoordeling van de pull-aanvraag met de volgende opmerking: Nee, opslagwachtrijen moeten kleine letters hebben.
Selecteer Beantwoorden & oplossen om aan te geven dat de discussie op de regel voorbij is.
De pull-aanvraag voltooien
Het ontwikkelteam van uw website heeft bevestigd dat het gereed is voor u om de orders naar de wachtrij te verzenden, zodat u klaar bent om uw pull-aanvraag te voltooien en samen te voegen.
Uw pull-aanvraag is goedgekeurd. Het ontwikkelteam van uw website heeft bevestigd dat het gereed is voor u om de orders naar de wachtrij te verzenden, zodat u klaar bent om uw pull-aanvraag te voltooien en samen te voegen.
Selecteer Pull-aanvraag samenvoegen.
GitHub vraagt u om de samenvoegbewerking te bevestigen. Wanneer GitHub de pull-aanvraag samenvoegt, wordt er een doorvoer gemaakt en wordt automatisch een doorvoerbericht gegenereerd. Selecteer Bevestig samenvoegen.
Uw pull-aanvraag wordt samengevoegd en uw nieuwe functie bevindt zich nu in de hoofdbranch van uw opslagplaats.
Het is een goed idee om uw functievertakkingen te verwijderen wanneer u er klaar mee bent. Door vertakkingen te verwijderen, voorkomt u in de toekomst verwarring onder de teamleden over welk werk nog in uitvoering is. Selecteer Verwijder vertakking.
Selecteer voltooien.
Gebruik in "Voltooi pull-aanvraag ",de standaardinstellingen. Selecteer Samenvoegen voltooien.
Uw pull-aanvraag wordt samengevoegd en uw nieuwe functie bevindt zich nu in de hoofdbranch van uw opslagplaats.
Azure DevOps heeft de functiebranch automatisch verwijderd toen u de pull-aanvraag hebt samengevoegd. Het is een goed idee om uw functievertakkingen te verwijderen wanneer u er klaar mee bent. Door vertakkingen te verwijderen, voorkomt u dat teamleden in de toekomst in de war raken over welk werk nog gaande is.
De wijzigingen controleren
Nadat u een pull-aanvraag hebt samengevoegd, is het een goed idee om te bevestigen dat de wijzigingen zijn samengevoegd.
Ga naar Code.
Ga naar het bestand deploy/main.bicep en ga vervolgens naar het bestand deploy/modules/appService.bicep.
U ziet dat de wachtrij en uw andere wijzigingen zich nu in de bestanden bevinden.
Ga naar repositories>bestanden.
Ga naar het bestand deploy/main.bicep en ga vervolgens naar het bestand deploy/modules/appService.bicep.
U ziet dat de wachtrij en uw andere wijzigingen zich nu in de bestanden bevinden.