Run linters on test scripts

[MichaelChirico]

The default pattern for files to lint is pattern = "(?i)[.](r|rmd|qmd|rnw|rhtml|rrst|rtex|rtxt)$", i.e. not Rraw. We don't need the other extensions, so trim them also for simplicity.

See:

https://lintr.r-lib.org/reference/lint.html

[MichaelChirico]

• Run linters on test scripts #6121 👈
• New linter for test(symbol) #6120
• customize lengths_linter for data.table usage #6100
• get code for custom linters working #6099
• Use R 3.2.0 features #5838
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MichaelChirico and the rest of your teammates on Graphite

[tdhock]

another option would be to change the Rraw to R as was discussed somewhere else?

[MichaelChirico]

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.

[tdhock]

ok great

[MichaelChirico]

Merge activity

• May 4, 12:06 PM EDT: @MichaelChirico started a stack merge that includes this pull request via Graphite.
• May 4, 12:20 PM EDT: Graphite rebased this pull request as part of a merge.
• May 4, 12:21 PM EDT: @MichaelChirico merged this pull request with Graphite.