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

Update vale styles. #1830

Merged
merged 3 commits into from Feb 3, 2021
Merged

Update vale styles. #1830

merged 3 commits into from Feb 3, 2021

Conversation

yhontyk
Copy link

@yhontyk yhontyk commented Feb 1, 2021

Read our Contribution guide before submitting a PR.

What does this PR do?

accept.txt specifies the capitalization and form of the word that we accept in the project. Some of the items in CommonTerms duplicate it. Cleaned up the styles, added regex definitions of certain words to fix bugs.

Also, disabling Vale.Repetition rule since it cretas a lot of false positives in image and include elements. Can't figure out how to escape those for now. =\

What issues does this PR fix or reference?

Specify the version of the product this PR applies to.

Current

PR Checklist

As the author of this Pull Request I made sure that:

  • vale has been run successfully against the PR branch
  • Link checker has been run successfully against the PR branch
  • Documentation describes a scenario that is already covered by QE tests, otherwise an issue has been created and acknowledged by Che QE team
  • Changed article references are updated where they are used (or a redirect has been set up on the docs side):

@themr0c themr0c self-assigned this Feb 3, 2021
@themr0c themr0c merged commit ce83969 into eclipse-che:master Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants