Breid regel "trailing slashes" uit - #353
Conversation
TimvdLippe
left a comment
There was a problem hiding this comment.
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 >.<
| <span id="api-48"></span> | ||
| <div class="rule" id="/core/no-trailing-slash" data-type="technical"> |
There was a problem hiding this comment.
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
| <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"> |
There was a problem hiding this comment.
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
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: