fix: add missing <Document> wrapper

[matijs]

No description provided.

[markacato]

Neem de review even over van @Robbert als je het niet erg vindt @matijs 🙂

[markacato]

Ah, nvm, @Robbert moet sowieso nog approven als requester natuurlijk 😊

[matijs]

Ah, nvm, @Robbert moet sowieso nog approven als requester natuurlijk 😊

Het maakt in principe niet uit wie approved. Er is alleen minimaal 1 approval nodig (en checks moeten passen natuurlijk).

[markacato]

Ah, nvm, @Robbert moet sowieso nog approven als requester natuurlijk 😊

Het maakt in principe niet uit wie approved. Er is alleen minimaal 1 approval nodig (en checks moeten passen natuurlijk).

@matijs dat dacht ik ook maar merging is alsnog blocked: "Merging can be performed automatically with 1 approving review."

[matijs]

Dat klopt want er is nu niemand die approved heeft.

Eerdere approvals worden automatisch dismissed als er nieuwe commits aan een PR worden toegevoegd of als er ge-force-pushed wordt.

Zie hierboven:

matijs dismissed markacato’s stale review via e0b5016 12 hours ago

Hiermee voorkomen we dat iemand (per ongeluk) iets aan een PR toevoegt dat dan zonder review gemerged kan worden.

[markacato]

@matijs zie nu inderdaad dat de review dismissed was 😊