-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: adds vale rules and workflow for running in CI #2348
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great and works both locally and in CI. Thanks for doing this!
We could add the "Redoc" but not "ReDoc" thing to the Vale rules so that it gets automatically caught if this mistake creeps back in. |
- [Pull Request Guidelines](#pull-request-guidelines) | ||
- [Development Setup](#development-setup) | ||
- [Commonly used NPM scripts](#commonly-used-npm-scripts) | ||
- [Project Structure](#project-structure) | ||
|
||
## Issue Reporting Guidelines |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I notice everywhere else in our docs and in our style guide we use sentence-case headings. Should this be adjusted here?
- [x-logo example](#x-logo-example) | ||
- [Tag Object](#tag-object) | ||
- [x-traitTag](#x-traittag) | ||
- [How to use with Redoc](#how-to-use-with-redoc-3) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a test (plz ignore)
Test, just ignore it. |
* docs: adds vale rules and workflow for running in CI * docs: updates product name from ReDoc to Redoc
What/Why/How?
In an effort to create consistently coherent documentation across our products we are going to apply the same Vale rules to all of the product doc repos. We have already applied these rules to the redocly CLI docs.
Check yourself