insightsengineering / insightsengineering/r.pkg.template

Proposal: Only run `urlchecker` on the built R package tarball

Open
#253 4 comments 0 reactions 0 assignees View on GitHub
Dominant language
Shell
Stars
43
Forks
17
PR merge metrics
No merged PRs in 30d

Description

Motivated by https://github.com/r-lib/urlchecker/issues/45#issuecomment-2363632879

I think we would not want `urlchecker` to return false alarms from files that are not actually part of the R package, but just of the repository. Therefore I would propose to only run `urlchecker` on the built R package tarball, not on the overall directory as it is currently done.

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.