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 R linting #440

Closed
maxheld83 opened this issue Feb 23, 2015 · 4 comments
Closed

add R linting #440

maxheld83 opened this issue Feb 23, 2015 · 4 comments

Comments

@maxheld83
Copy link

that would be fantastic ...

@hd-deman
Copy link
Contributor

@maxheld83 r u talking about this tool?

@hd-deman
Copy link
Contributor

I also find this http://yihui.name/formatR/

@maxheld83
Copy link
Author

@hd-deman thanks for getting back so quickly.
I was thinking of @jimhester's lintr package, which I find quite good.
I haven't systematically compared R linters though, I just came across (and liked) lintr, because it is already implemented for Sublime Text.

I guess formatR would be a different beast, since that actually changes (rather than lints) code.

@jimhester
Copy link

lintr has nearly all of the functionality of the existing R linting packages, (lint and svtools were the main ones I was aware of), plus many more.

r-lib/lintr#42 is for adding Atom integration (we already have Vim, Emacs, Sublime Text, RStudio).

It shouldn't be too much work to get Atom integration working if you base it off of the existing integrations.

I haven't done so yet because I do not use Atom myself and I wasn't sure how popular it was for R users, but at least different people have now requested it, so it is probably worthwhile.

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

5 participants