Skip to content

Patch linter configuratie#343

Merged
TimvdLippe merged 3 commits into
mainfrom
patch-linter-configuratie
Jun 18, 2026
Merged

Patch linter configuratie#343
TimvdLippe merged 3 commits into
mainfrom
patch-linter-configuratie

Conversation

@TimvdLippe

Copy link
Copy Markdown
Contributor

No description provided.

TimvdLippe and others added 3 commits June 18, 2026 13:44
Hiermee zijn er geen warnings meer. Dit maakt het mogelijk
voor developer.overheid.nl om warnings te genereren voor
toekomstige linter versies. Ze willen namelijk regels die nieuw zijn
in versie 2.X toevoegen als warnings aan versie 2.(X - 1).
Er is een rule voor het gebruik van camelCase
van query keys: `nlgov:query-keys-camel-case`.
Ik ging er onterecht vanuit dat deze regel ook
hiervoor is.

Dat is echter niet zo, want deze regel gaat over
het beschrijven van schema's in een OpenAPI
specificatie. Dat is niet als regel in de ADR
opgenomen en moeten we daarom ook niet
checken.
@TimvdLippe TimvdLippe requested a review from sanderke June 18, 2026 11:47
@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 Jun 18, 2026
@github-actions

Copy link
Copy Markdown

@github-actions github-actions Bot added Status: Ter goedkeuring Het voorstel is uitgewerkt en wordt ter goedkeuring aangeboden. and removed Status: In bewerking Het voorstel is in bewerking bij de beheerorganisatie. labels Jun 18, 2026
@TimvdLippe TimvdLippe merged commit 3472ec6 into main Jun 18, 2026
17 checks passed
@TimvdLippe TimvdLippe deleted the patch-linter-configuratie branch June 18, 2026 12:51
@github-actions github-actions Bot added Status: Klaar voor release Het voorstel is verwerkt en klaar voor de volgende release. and removed Status: Ter goedkeuring Het voorstel is uitgewerkt en wordt ter goedkeuring aangeboden. labels Jun 18, 2026
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: Klaar voor release Het voorstel is verwerkt en klaar voor de volgende release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants