Skip to content

Beheer: haal property casing schema rule weg#342

Merged
TimvdLippe merged 1 commit into
developfrom
TimvdLippe-patch-1
Jun 18, 2026
Merged

Beheer: haal property casing schema rule weg#342
TimvdLippe merged 1 commit into
developfrom
TimvdLippe-patch-1

Conversation

@TimvdLippe

Copy link
Copy Markdown
Contributor

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.

Fixes #340

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 17, 2026 07:05
@github-actions

Copy link
Copy Markdown

@TimvdLippe TimvdLippe merged commit e1b6294 into develop Jun 18, 2026
16 of 17 checks passed
@TimvdLippe TimvdLippe deleted the TimvdLippe-patch-1 branch June 18, 2026 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rules voor CamelCase schemas en lowerCamelCase properties missen

2 participants