Skip to content

Conversation

sg-writer
Copy link
Contributor

@sg-writer sg-writer commented Oct 2, 2025

this PR installs Vale with (most of) the Google style rules package to get us started. I made some changes throughout to make it more useful to us but there's definitely more to be done. This should be enough to start using Vale right away as a CI check.

I tried getting an LLM to fix the 500ish errors reported but it made a colossal mess so I had to revert all of those changes. We can start to actually apply the Vale rules in follow-up PRs.

The command to run it is pnpm run vale. There are lots of flags available to refine it.

Note: once this PR is merged we'll need to turn Vale back on in the Mintlify dashboard to get the CI checks.

"verify-redirects": "node snippets/custom-scripts/check-redirect-conflicts.js",
"test-links": "mint broken-links"
"test-links": "mint broken-links",
"vale": "vale --glob='!{snippets,errors}/**' ."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it possible to add vale to the dependencies? When I try to run this command I get the error that vale is not installed

Also, we should mention this command in the readme

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs/infra Changes to the infrastructure of the docs site kind/enhancement New feature or improvement to the docs experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants