-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
[R-package] Add additional linters to CI #2477
Comments
Closing in favor of being in #2302 with other feature requests. If you'd like to submit a PR to address this, re-open the issue. |
Since we use |
it seems that lintr did a release on October 1, 2019! I am going to re-open this issue and pick it up tonight or tomorrow. |
and they updated the conda feedstock for r-lintr 🙌 Seems that project has a bot that automatically opens a PR when a new version makes it to CRAN (or maybe when a new release is created on Either way, should be able to address this issue now! |
Linting for the R code in this project was introduced in #2437 . As of that PR, the following linters from the
lintr
project were not available from the distribution oflintr
on CRAN.To close this issue, add those linters to
.ci/lint_r_code.R
oncelintr
does a new release (> 1.0.3) to CRAN.The text was updated successfully, but these errors were encountered: