Box import separate packages modules

[radbasa]

Changes

• Closes Box import packages separate from modules #541
• Adds a box_separate_calls_linter()
• Adds the litner to the .lintr template
• Adds documentation

How to test

1. Unit tests for box_separate_calls_linter() exist.
2. Locally install rhino, version should be 1.6.0.9004, initialize a new rhino application then proceed to declare box imports.
3. Add a box::use() with both package and modules.
4. rhino::lint_r() should find and point to the invalid box::use() call.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b4722ba) 31.64% compared to head (be25e88) 34.11%.

❗ Current head be25e88 differs from pull request most recent head 3a30d82. Consider uploading reports for the commit 3a30d82 to get more accurate results

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

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #542      +/-   ##
==========================================
+ Coverage   31.64%   34.11%   +2.46%     
==========================================
  Files          10       10              
  Lines         534      554      +20     
==========================================
+ Hits          169      189      +20     
  Misses        365      365              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jakubnowicki]

Linter is not able to detect problems if the package imports functions:

box::use(
  path/to/module,
  package[function1, function2],
)

[radbasa]

Linter is not able to detect problems if the package imports functions:

box::use(
  path/to/module,
  package[function1, function2],
)

Fixed.

module and module[function] have different XML patterns.

Added unit tests for aliased modules and functions. Just in case.

[jakubnowicki]

LGTM