-
Notifications
You must be signed in to change notification settings - Fork 82
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: add spell check capabilities to Vale linter #1345
base: master
Are you sure you want to change the base?
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.
As long as the CI is happy, I am happy 🙃
remove typos workflow & config file create extensive accept.txt that catches known exceptions add vale turn off in problematic places until better solution is found
add datacenter to accept.txt fix issues with mdlint on milvus integration page remove unnecessary vale off & on comments due to fixing accept & reject vocabularies
@fnesveda , @mtrunkat This should help with typos throughout PR's. While our previous tool started strong (catching over 100 typos), it eventually started missing some pretty obvious ones. Vale is much more reliable ,with much bigger dictionary, and will actually block merges until typos are fixed. Plus, we can make it even smarter by adding our own regex-based rules to |
No description provided.