Skip to content

Beheer: verplaats linter configuratie#307

Merged
TimvdLippe merged 1 commit intodevelopfrom
verplaats-linter-configuratie
Mar 2, 2026
Merged

Beheer: verplaats linter configuratie#307
TimvdLippe merged 1 commit intodevelopfrom
verplaats-linter-configuratie

Conversation

@TimvdLippe
Copy link
Contributor

Hiermee bereiken we twee doelen:

  1. De naamgeving is niet gelimiteerd aan Spectral, omdat ook andere tooling deze configuratie kan gebruiken
  2. Omdat het nu in media/ staat, publiceren we deze configuratie nu ook op een aparte plek. Hierdoor is het mogelijk om voor andere tooling om automatisch het bestand te pakken, in plaats van die uit een bijlage van het standaarddocument te halen

Uiteindelijk zorgen we ervoor dat tooling zoals developer.overheid.nl nu naar deze configuratie kunnen verwijzen en ook automatisch up-to-date zijn met wijzigingen die we aanbrengen.

Fixes #254

@TimvdLippe TimvdLippe requested review from dvh and sanderke March 2, 2026 10:53
@github-actions
Copy link

github-actions bot commented Mar 2, 2026

Hiermee bereiken we twee doelen:
1. De naamgeving is niet gelimiteerd aan Spectral, omdat ook andere
tooling deze configuratie kan gebruiken
2. Omdat het nu in `media/` staat, publiceren we deze configuratie
nu ook op een aparte plek. Hierdoor is het mogelijk om voor andere
tooling om automatisch het bestand te pakken, in plaats van die
uit een bijlage van het standaarddocument te halen

Uiteindelijk zorgen we ervoor dat tooling zoals developer.overheid.nl
nu naar deze configuratie kunnen verwijzen en ook automatisch
up-to-date zijn met wijzigingen die we aanbrengen.
@TimvdLippe TimvdLippe force-pushed the verplaats-linter-configuratie branch from a074c55 to 25a7e4c Compare March 2, 2026 13:32
@TimvdLippe
Copy link
Contributor Author

@dvh als we dit zouden mergen, dan zou de configuratie staan op https://gitdocumentatie.logius.nl/publicatie/api/adr/2.1.0/media/linter.yml en kunnen jullie die gebruiken in jullie tooling API.

@TimvdLippe TimvdLippe merged commit 056a1aa into develop Mar 2, 2026
16 checks passed
@TimvdLippe TimvdLippe deleted the verplaats-linter-configuratie branch March 2, 2026 13:57
@dvh
Copy link
Collaborator

dvh commented Mar 3, 2026

Zou dit automatisch gepubliceerd moeten worden? Want ik krijg hier nog een 404: https://gitdocumentatie.logius.nl/publicatie/api/adr/2.1.0/media/linter.yml

@TimvdLippe
Copy link
Contributor Author

Het zou automatisch moeten gaan, maar we hebben synchronisatie problemen. We proberen dat op te lossen, geen idee wanneer ons dat gaat lukken.

@TimvdLippe
Copy link
Contributor Author

Overigens wordt de uiteindelijke URL waarschijnlijk https://gitdocumentatie.logius.nl/publicatie/api/adr/2.1.0/media/linter.yaml (dus .yaml ipv .yml), maar door de synchronisatieproblemen hebben we dat nog niet kunnen verifieren

@TimvdLippe
Copy link
Contributor Author

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.

Spectral linting rules vervangen met een alternatief

3 participants