box_usage_linter

[radbasa]

Changes

Packages

• lints on function not attached
• lints on package$function not exists
• lints on attached function unused
• lints on attached package unused
• lints on invalid function from package box::use(shiny[no_func])
• handles box::use(package[function])
• handles box::use(package)
• handles box::use(package[...])
• is aware of base packages

Local functions

• lints on declared function unused - may have to remove this. our box modules are libraries of function declarations!
• handles functions declared in source file
• handles internal R6 class object calls

Box modules

TODO

• roxygen docs
• will need lots of QA for this (and test cases)

  Packages

Local functions

• ~handle S4 setMethod() attachment~ Will not do. box and S4 do not play well together.
• handle box aliases

  Box modules

• handle file module box attachment

How to test

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 59.26%. Comparing base (0ff3c44) to head (311de91). Report is 18 commits behind head on main.

:exclamation: Current head 311de91 differs from pull request most recent head 1172732. Consider uploading reports for the commit 1172732 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #571 +/- ## =========================================== + Coverage 34.11% 59.26% +25.14% =========================================== Files 10 17 +7 Lines 554 896 +342 =========================================== + Hits 189 531 +342 Misses 365 365 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[radbasa]

@jakubnowicki I added

  box::use(
    path/to/module1,
    path/to/module2[a, b, c],
    path/to/module3[...]
  )

to the test cases where it makes sense. All tests just worked as expected. When you have time, please take a look. It was a surprise for me.