Add --load-package to lintr

[agvandervelde]

Hi @lorenzwalthert, I need to load packages prior to running lintr in CI/CD, so I can account for "spurious" global variables (e.g., when using box, dplyr, etc.) using utils::globalVariables() (see r-lib/lintr#352 (comment)). I'm currently running off of the branch on my fork but I think it'd be useful to incorporate this in your upstream. Issue #440 suggests there's a broader need for this.

[lorenzwalthert]

Hi @agvandervelde , do I understand correctly that you list all your package's dependencies in additional_dependenies, just like for the roxygenise hook? Or do you override language: r? In any case, that needs to be documented better. I'll try to do amend your PR to that effect.

[agvandervelde]

Yes, I list the dependencies in additional_dependenies. We're doing that already for styler so it's not an additional burden in our case. Thank you for looking into this!

[lorenzwalthert]

For styler? I don’t get it. How would the styler hook need additional dependencies? You mean roxygen? This needs to load the package as per the docs.

[agvandervelde]

For styler? I don’t get it. How would the styler hook need additional dependencies? You mean roxygen? This needs to load the package as per the docs.

Apologies, yes, roxygen.

[lorenzwalthert]

Hi @agvandervelde, sorry this took a while... I pushed some (mildly related) doc changes to this branch too and I think this should be mergeable soon. Thanks for your work.

[agvandervelde]

Thank you!

[agvandervelde]

Rebased this PR onto current main.

[lorenzwalthert]

Sorry did not get to merge this yet. Hope everything passes now.

[lorenzwalthert]

@agvandervelde thanks again. If you don’t want to wait till the next tag bump, you can also pre-commit autoupdate --bleeding-edge or similar to use the lastest main commit.