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

Add rubocop linter #3

Merged
merged 3 commits into from
Sep 15, 2016
Merged

Add rubocop linter #3

merged 3 commits into from
Sep 15, 2016

Conversation

ynonp
Copy link
Contributor

@ynonp ynonp commented Sep 15, 2016

For ruby projects add support for rubocop linter using "ruby" file type

endfunction

call ALEAddLinter('ruby', {
\ 'executable': 'rubocop',
Copy link
Member

Choose a reason for hiding this comment

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

With changes I have made, I'll need to add a name key here now, which is now required. I'll merge this and add it in. Could you test this again after I make that change?

@@ -0,0 +1,45 @@
if exists('g:loaded_ale_linters_ruby_rubocop')
Copy link
Member

Choose a reason for hiding this comment

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

I'll add Author comments to the top of this file and others to credit you. Let me know if there is a preferred name you would like in the files via email or just comments on GitHub.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool! just use my github username ynonp

@w0rp w0rp merged commit 36b25a5 into dense-analysis:master Sep 15, 2016
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.

None yet

2 participants