insightsengineering / insightsengineering/r.pkg.template

[Feature Request]: linter check should allow for `object_usage_linter` not being disabled

Open
#299 0 comments 0 reactions 0 assignees View on GitHub
enhancement
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

Image

### 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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.