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

Glue for Vale #522

Merged
merged 3 commits into from
May 1, 2017
Merged

Glue for Vale #522

merged 3 commits into from
May 1, 2017

Conversation

chew-z
Copy link
Contributor

@chew-z chew-z commented Apr 30, 2017

Following discussion here I have added simple glue for vale linter for markdown and text.

Vale is cool linter for natural language and it works well with ale. The only trick is to use option --output=line to adjust to ale#handlers#unix#HandleAsWarning function.

@w0rp
Copy link
Member

w0rp commented Apr 30, 2017

Have a look at the Travis output. It's complaining about your double-quoted strings. I recommend installing vint, which ALE integrates with.

Copy link
Contributor Author

@chew-z chew-z left a comment

Choose a reason for hiding this comment

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

Changed double quotes to single quotes.

@chew-z
Copy link
Contributor Author

chew-z commented Apr 30, 2017

I have vint installed but mine wasn't complaining about double quotes. Anyway I have changed to single quotes and it works.

@w0rp w0rp merged commit d5c135c into dense-analysis:master May 1, 2017
elebow pushed a commit to elebow/ale that referenced this pull request May 3, 2017
Add Vale linter definitions
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