Skip to content
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

Added Hunspell en_US dictionary to Vale configuration #94

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

geoffreynyaga
Copy link

This PR integrates the Hunspell en_US dictionary into Vale to enhance spell-checking capabilities.

Changes:

  • Added recommended Hunspell en_US dictionary to the Vale configuration. repo

To be advised:

  • The styleguide YAML files in styles/Canonical/* have an existing numbering convention. Should we do the same to the en_US_spellcheck.yaml file?
  • Test and verify that the dictionary integration works as expected with Vale on CI.

Copy link
Contributor

@SecondSkoll SecondSkoll left a comment

Choose a reason for hiding this comment

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

Actually, could you rename this rule to 000-US-spellcheck.yaml add this rule to vale.ini file in the root with:

Canonical.000-US-spellcheck = error

Just for consistency and so it doesn't run with the style guide check that is currently available.

We'll have to do some testing before this is ready, and it'll be provided with a filter so spelling, terminology, and style checks are all separate.

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