Hoe ik Git gebruikte
Voor versiebeheer gebruikte ik Git met een remote in Azure DevOps. Ik werkte op een taakbranch en maakte een aparte branch als grotere werkzaamheden los van de endpoints moesten gebeuren. In mijn oorspronkelijke tekst beschrijf ik pull requests richting Dev1. De codebranches in het archief heten onder andere task/initialsetup, task/15264_phOUnix en develop; Dev1 is ook de naam van een omgeving. Die twee haal ik daarom niet door elkaar.
Voorbeelden uit het aangeleverde Git-archief
| Datum | Commit | Wat de omschrijving laat zien |
|---|---|---|
| 18 november 2025 | 930692b | Generiek mapping service |
| 25 november 2025 | f4b5286 | Authorization werkt |
| 8 december 2025 | a4ecfca | pageSize op basis van enum; 400 bij onjuiste query |
| 26 januari 2026 | 2f9bf11 | v5 models verwijderen |
| 4 februari 2026 | 0546727 | Product mapping |
| 4 maart 2026 | b7961dd | sku naar level |
| 8 april 2026 | 5ced487 | bicep 3 |
Deze commits zijn echt uit .git in OOAPI.zip gelezen. De omschrijving laat zien waar een wijziging over ging; de bijbehorende diff is nodig om de exacte verandering te beoordelen. Een commit met “werkt” is op zichzelf geen testresultaat.
Wat ik ervan leer
Branches houden werkzaamheden apart en met commits kan ik een eerdere versie terugvinden. Omschrijvingen zoals “bicep 3” zijn achteraf minder duidelijk. Een volgende keer zou ik de taak en de concrete wijziging noemen, bijvoorbeeld “Pas Bicep-configuratie voor ontwikkelomgeving aan”. Dat voorbeeld is een voorstel, geen bestaande commit.
Deze commitlijst laat wijzigingen zien, maar geen review van een pull request.
Eén wijziging van dichtbij
In de code heb ik ook één wijziging nagekeken. Commit a4ecfca van 8 december 2025 voegt Core/Enums/PageSize.cs en Retriever/Services/EnumValidator.cs toe. De enum bevat 10, 20, 50, 100 en 250. De validator accepteert een ontbrekende waarde met de standaardinstelling, controleert numerieke invoer met Enum.IsDefined en geeft voor overige invoer false terug.
Daarmee kan ik de wijziging achter de commitomschrijving laten zien. pageSize=1 hoort volgens deze code bij de negatieve tests. TC-10 beschrijft de controle op een 400-response. Deze diff bewijst geen code-review of geslaagde uitvoering.
De wijziging is lokaal terug te vinden met:
git show a4ecfca -- src/OU.HIP.OOAPI.Core/Enums/PageSize.cs src/OU.HIP.OOAPI.Retriever/Services/EnumValidator.cs