dfe-analytical-services/dfeshiny

lintr action is incorrectly flagging missing bindings

Closed this issue · 1 comments

I've noticed on PRs that the lintr workflow seems to be incorrectly flagging bindings as missing, when they're clearly imported in the scripts. Has happened on #3.

image

Recreating it locally seems like it's missing a step in the workflow. If you run devtools::load_all() before running lintr::lint_package() it seems to fix it.

Would be good to resolve / fix to clear out the clutter it can bring, as some of these functions like tags() can use many many times across the package.

I think this may be fixed by #41