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.

Sign up for GitHub

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

Jump to bottom

Refine built-in rules #30

Closed
jdkato opened this issue Mar 22, 2017 · 0 comments
Closed

Refine built-in rules #30

jdkato opened this issue Mar 22, 2017 · 0 comments
Labels
Type: Refactor

Comments

@jdkato
Copy link
Member

jdkato commented Mar 22, 2017 •
edited
Loading

I think Vale should ship with a minimal set of commonly used and/or non-subjective rules. Considering this, I think:

  • PassiveVoice should be decreased to a suggestion.
  • MinAlertLevel should default to warning (it's currently suggestion).
jdkato added a commit that referenced this issue Mar 23, 2017
@jdkato
refactor: move Abbreviations to the demo style
1f3a3e9 
Related to #30
@jdkato jdkato mentioned this issue Mar 27, 2017
Closed
jdkato added a commit that referenced this issue Mar 31, 2017
@jdkato
refactor: make warning the default level
272052e 
Also, demotes `Annotations` and `PassiveVoice` to `suggestions`.

Related to #30.
@jdkato jdkato closed this as completed Mar 31, 2017
@jdkato jdkato mentioned this issue May 16, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Refactor
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jdkato