Sta relatieve Server API URL toe.#291
Sta relatieve Server API URL toe.#291kad-sprenr wants to merge 2 commits intoLogius-standaarden:developfrom
Conversation
github-actions
bot
added
Status: In bewerking
TimvdLippe
left a comment
TimvdLippe
left a comment
There was a problem hiding this comment.
Zoals beschreven in #290 (comment) moeten we eerst een design regel hierover definiëren (of iig de discussie of we dat moeten doen) voordat we de linter configuratie kunnen aanpassen.
linter/spectral.yml
Outdated
| function: pattern | ||
| functionOptions: | ||
| match: ^https://.* | ||
| match: ^(https://.*|/.*) |
There was a problem hiding this comment.
Er zijn ook andere soorten relatieve URLs:
//...(protocol-relatief)../...ofsegment/...(relatieve paden)
Die laatste wordt door deze regex niet toegestaan.
There was a problem hiding this comment.
Aangepast. Even los van de discussie of je al die verschillende vormen wilt toestaan.
|
@TimvdLippe Laat vooral weten hoe je verder wilt gaan met deze PR n.a.v. het TO. |
|
Gisteren heb ik #294 geupdate en daar ook een fix in meegenomen. Afhankelijk van de beslissing op het TO wordt ofwel het gehele pakket of deels gemergt. We zorgen er sowieso voor dat na het TO jouw issue wordt voorkomen, ongeacht de uiteindelijke beslissing of er wel/niet een regel voor moet komen. |
3 participants
Zie #290