insightsengineering / insightsengineering/r.pkg.template
[Feature Request]: linter check should allow for `object_usage_linter` not being disabled
- Dominant language
- Shell
- Stars
- 43
- Forks
- 17
- PR merge metrics
- No merged PRs in 30d
Description
### Feature description
It appears that the linter CI detects all internal functions and imports as valid objects.
We have ignored this linter as a rule, but I believe we need it as there are unused items and even found a silent bug that would be detected by this.
I would prefer that the CI would support this _(at first glance we will probably need to install the dependencies and load package prior to lint (which will consume time and resources)_
Alternatively, we could ensure that the rule is disabled for CI, but still enabled for developers locally.
See here a run in the `teal.picks` package with linter disabled:
https://github.com/insightsengineering/teal.picks/actions/runs/25155870362/job/73737413211
### Code of Conduct
- [x] I agree to follow this project's Code of Conduct.
### Contribution Guidelines
- [x] I agree to follow this project's Contribution Guidelines.
Contributor guide
Assessment
This issue has not been assessed yet.