-
Notifications
You must be signed in to change notification settings - Fork 985
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
Run linters on test scripts #6121
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @MichaelChirico and the rest of your teammates on Graphite |
66bc2ab
to
f928431
Compare
077f4d1
to
0c8d1e7
Compare
another option would be to change the Rraw to R as was discussed somewhere else? |
yes! I had a TODO at some point but must have lost it. That issue won't be fixed for some time I believe, until we burn down most of the existing PR queue. |
ok great |
00b1406
to
1d045f8
Compare
7273b4f
to
5abbc1d
Compare
Merge activity
|
1d045f8
to
ed0d93d
Compare
5abbc1d
to
35c44e3
Compare
The default pattern for files to lint is
pattern = "(?i)[.](r|rmd|qmd|rnw|rhtml|rrst|rtex|rtxt)$"
, i.e. notRraw
. We don't need the other extensions, so trim them also for simplicity.See:
https://lintr.r-lib.org/reference/lint.html