rhino icon indicating copy to clipboard operation
rhino copied to clipboard

Box clearing mechanism (linter rule)

Open Leszek-Sieminski opened this issue 2 years ago • 0 comments

Motivation

There's no mechanism that checks if packages/functions included in box::use() are actually used in the file, which can lead to a situation where old dependencies still linger in the project despite not being used.

Feature description

I imagine that user could run rhino::linter_r() function that would have list the functions that are not used in the file.

Implementation ideas

new linter rule that would detect functions declared in box::use() calls and then list the functions that are not used in the context

Impact

It would be used with other linter rules, so basically it would be used in every commit push (via Github Actions CI).

If we don't implement it there's no immediate danger but we are leaving a sort of project vulnerability - projects may grow their dependencies without developer's control and this may endanger stability in the long run. So I'd say it's worth considering.

Leszek-Sieminski avatar Nov 23 '22 14:11 Leszek-Sieminski