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

Support linting R #210

Open
davidchall opened this issue May 24, 2018 · 3 comments
Open

Support linting R #210

davidchall opened this issue May 24, 2018 · 3 comments

Comments

@davidchall
Copy link

Would it be possible to add support for linting R code? This might be achieved using https://github.com/jimhester/lintr

@markstory
Copy link
Owner

I'm not familiar with R at all, so I don't think I'd be a good person to implement the lintr integration. I am happy to help with the python/docker pieces if you know how to get lintr running, and have sample files that can be used for regression testing the tool adapter.

I wasn't able to determine from the lintr docs whether or not there was a checkstyle xml output, or a emacs/vim quickfix output option. If there adding a tool adapter will be much simpler.

@louiscpalma
Copy link

Just happened across this project while looking into modifying an in-house lint bot:

lintr can generate checkstyle reports with the checkstyle_output() function.

Page 4 of the pdf manual: https://cran.r-project.org/web/packages/lintr/lintr.pdf

Probably wouldn't integrate as nicely as ansible-lint -p, but should be manageable.

@markstory
Copy link
Owner

@louiscpalma How does one go about running lintr on one or more files? I've never used R before 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants