Skip to content

feat: add linting - #14

Merged
yeoldegrove merged 1 commit into
mainfrom
feat/linting
Mar 26, 2026
Merged

yeoldegrove merged 1 commit into
mainfrom
feat/linting

Conversation

@yeoldegrove

@yeoldegrove yeoldegrove commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Add some initial linting:

  • spellcheck via codespell
  • link check via lychee
  • inclusive language via woke

@netlify

netlify Bot commented Mar 26, 2026

Copy link
Copy Markdown

Deploy Preview for gardenlinux-docs ready!

Name Link
🔨 Latest commit d4e96c8
🔍 Latest deploy log https://app.netlify.com/projects/gardenlinux-docs/deploys/69c4f76fefe4b300088b003f
😎 Deploy Preview https://deploy-preview-14--gardenlinux-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@yeoldegrove
yeoldegrove force-pushed the feat/linting branch 11 times, most recently from 3390bdd to 0c91e3e Compare March 26, 2026 09:07
Signed-off-by: Eike Waldt <waldt@b1-systems.de>
On-behalf-of: SAP <eike.waldt@sap.com>
@ByteOtter

Copy link
Copy Markdown
Collaborator

woke is a cool addition, but it seems to me that the project is abandoned and has some licensing issues? Just pointing this out in case this would touch us as well.

@yeoldegrove

Copy link
Copy Markdown
Contributor Author

woke is a cool addition, but it seems to me that the project is abandoned and has some licensing issues? Just pointing this out in case this would touch us as well.

Thanks for pointing this out... The ubuntu server CI is using it, this is where I found out about it.

@ByteOtter ByteOtter left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

As long as the mentioned license issues with woke doesn't touch us, I think this is good to go right now.

@yeoldegrove

Copy link
Copy Markdown
Contributor Author

Yes, let's find another project later that has the same features.

@yeoldegrove
yeoldegrove merged commit 0532763 into main Mar 26, 2026
5 of 7 checks passed
@yeoldegrove
yeoldegrove deleted the feat/linting branch April 15, 2026 15:09
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.

2 participants