Skip to content

Breid regel "trailing slashes" uit - #353

Open
sanderke wants to merge 2 commits into
developfrom
trailing-slash-redux
Open

Breid regel "trailing slashes" uit#353
sanderke wants to merge 2 commits into
developfrom
trailing-slash-redux

Conversation

@sanderke

@sanderke sanderke commented Aug 11, 2026

Copy link
Copy Markdown
Member

Uitgewerkt ingezonden voorstel om empty path segments te vermijden in het algemeen in plaats van enkel aan het einde van een URI.

Resolves #349

To do:

  • Update linter

@github-actions github-actions Bot added Status: In bewerking Het voorstel is in bewerking bij de beheerorganisatie. Overleg: TO-API Te agenderen voor het Technisch Overleg API labels Aug 11, 2026
@github-actions

Copy link
Copy Markdown

@TimvdLippe TimvdLippe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uitwerking lijkt mij goed. Ik ben benieuwd hoe het TO denkt over of dit een minor of major change is. Gezien het volgende:

Het wijzigen van bestaande design rules blijft een Major release.

in de RELEASING.md ga ik er vanuit dat het enkel een major change kan zijn. Dat vereist wat bundling van wijzigingen, anders is het wel zonde om enkel hiervoor een major change te maken.

We kunnen ook een extra regel definieren, want dan lopen we hier niet tegenaan >.<

Comment thread sections/designRules.md Outdated
Comment on lines 84 to 85
<span id="api-48"></span>
<div class="rule" id="/core/no-trailing-slash" data-type="technical">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik denk dat we dan het moeten hernoemen, maar we kunnen de links nog wel behouden door zo'n extra span toe te voegen. Dit moeten we dan ook veranderen in de linter configuratie

Suggested change
<span id="api-48"></span>
<div class="rule" id="/core/no-trailing-slash" data-type="technical">
<span id="api-48"></span>
<span id="/core/no-trailing-slash"></span>
<div class="rule" id="/core/no-empty-path-segments" data-type="technical">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dit moeten we dan ook veranderen in de linter configuratie

Kun je dit nog updaten? Nu is de link weggevallen in de tekst van "How to test": https://logius-standaarden.github.io/Publicatie-Preview/API-Design-Rules/trailing-slash-redux/#/core/no-empty-path-segments

Co-authored-by: Tim van der Lippe <TimvdLippe@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Overleg: TO-API Te agenderen voor het Technisch Overleg API Status: In bewerking Het voorstel is in bewerking bij de beheerorganisatie.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rule /core/no-trailing-slash uitbreiden (// ook uitsluiten)

2 participants