Skip to content

Conversation

@richadr
Copy link
Contributor

@richadr richadr commented Jan 14, 2026

Preview link

  • Is de redirect goed? Zit hier niet wat voor in docusaurus zelf?
  • Check slug en titel. "Linting" of "Linten"?
  • Toelichten noemenswaardige configuratie, bijvoorbeeld EUPL licentie in npmpkgjsonlint
  • user stories toevoegen

@changeset-bot
Copy link

changeset-bot bot commented Jan 14, 2026

⚠️ No Changeset found

Latest commit: d617647

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Jan 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
documentatie Ready Ready Preview, Comment Jan 28, 2026 9:28pm

Request Review

@richadr richadr force-pushed the docs/linting-and-formatting branch from a0d3141 to eff6140 Compare January 21, 2026 14:54
@richadr richadr marked this pull request as ready for review January 21, 2026 14:54
@richadr richadr requested a review from a team as a code owner January 21, 2026 14:54
@richadr richadr force-pushed the docs/linting-and-formatting branch from 6b7eaea to bfe0d76 Compare January 21, 2026 14:59
@richadr richadr enabled auto-merge (rebase) January 21, 2026 15:38
@eslook
Copy link
Contributor

eslook commented Jan 22, 2026

Gezien we ook deployen en builden doen, vind ik linten best consistent en leesbaar.

Echter past linten niet overal: linten is een werkwoord. Op dit moment gebruiken we linten niet, en linting op twee plekken:

  • docs/handboek/leverancier/definition-of-done: Codebase gebruikt linting regels
  • docs/handboek/leverancier: een licentie en linting regels

Daar vind ik linting leesbaarder dan linten. Linten is een werkwoord, linting is meer een proces of term.

Dus dan zou ik consistent altijd het woord linting gebruiken.

@eslook
Copy link
Contributor

eslook commented Jan 22, 2026

Docusaurus heeft client side redirect, maar server side redirect zoals je nu hebt via htaccess lijkt me netter.

@richadr richadr force-pushed the docs/linting-and-formatting branch from ad12106 to d617647 Compare January 28, 2026 21:25
@richadr richadr requested a review from Robbert January 28, 2026 21:25
@sonarqubecloud
Copy link

Comment on lines +105 to +107
- Schakel **format on save** in en configureer het om Prettier uit te voeren
- Gebruik dezelfde [Node.js‑versie](/nodejs) als het project
- Zorg dat de lokale Prettier config wordt gebruikt
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Schakel **format on save** in en configureer het om Prettier uit te voeren
- Gebruik dezelfde [Node.js‑versie](/nodejs) als het project
- Zorg dat de lokale Prettier config wordt gebruikt
- Schakel **format on save** in en configureer het om Prettier uit te voeren.
- Gebruik dezelfde [Node.js‑versie](/nodejs) als het project.
- Zorg dat de lokale Prettier config wordt gebruikt.

En vraagje: hoe zorg je dat de lokale Prettier config wordt gebruikt? Is dat vaak nodig om in te stellen?


## Voorbeeld en referentie

Een volledig voorbeeld van de configuratie en best practices wordt onderhouden in de [example](https://github.com/nl-design-system/example) repository.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Een volledig voorbeeld van de configuratie en best practices wordt onderhouden in de [example](https://github.com/nl-design-system/example) repository.
Gebruik de de configuratie voor linting en formatting in de de [example](https://github.com/nl-design-system/example) repository als startpunt voor nieuwe projecten. Kijk ook in deze repository voor up-to-date configuraties wanneer je upgrades installeert van de tools, om tijd te besparen.

Voorstel om de essentiele info aan het begin van de zinnen te zetten, in plaats van te beginnen met context.

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.

4 participants