Remove custom linting logic from addins

[MichaelChirico]

Closes #2525

Probably I don't understand how add-ins are invoked, but it seems this approach should work & is much simpler.

cc @HaHeho -- are you able to test if this works?

[HaHeho]

cc @HaHeho -- are you able to test if this works?

I would like to try. How can I install this version from within R Studio?

[MichaelChirico]

maybe remotes::install_github("r-lib/lintr@addin-config")? and be sure to close/restart all RStudio windows...

[AshesITR]

LGTM. Please wait for a positive user test just to be sure, though.

[HaHeho]

Apologies for the delay. I have successfully tested the fix (at least with my specific configuration).

Before update:

> packageVersion("lintr")
[1] ‘3.1.1’

> lintr::lint("file") # with a ".lintr.R"
# WORKS

> lintr:::addin_lint() # with a ".lintr.R"
Error in read.dcf(config_file, all = TRUE) : Invalid DCF format.
Regular lines must have a tag.
Offending lines start with: ...

After remotes::install_github("r-lib/lintr@addin-config") and a complete restart of RStudio:

> packageVersion("lintr")
[1] ‘3.1.1.9000’

> lintr::lint("file") # with a ".lintr.R"
# WORKS

> lintr:::addin_lint() # with a ".lintr.R"
# WORKS

With the new version, I receive a few more linter warnings and Trying to remove ‘extraction_operator_linter’, which is not in "defaults". (in my specific config). I assume this is intended and related to other linter additions with this version, as it is consistent from both the console and adding calls.

[MichaelChirico]

Yep, that's intended: 32cb18c