Skip to content

docs: silence devtools::document() roxygen link warnings#118

Merged
VincentGuyader merged 1 commit into
masterfrom
docs/roxygen-link-cleanup
May 13, 2026
Merged

docs: silence devtools::document() roxygen link warnings#118
VincentGuyader merged 1 commit into
masterfrom
docs/roxygen-link-cleanup

Conversation

@VincentGuyader
Copy link
Copy Markdown
Member

Summary

Silences the two long-standing devtools::document() warnings:

✖ utils.R:38: @description Could not resolve link to topic ".github_pat_run_prefix"
✖ utils.R:55: @description Could not resolve link to topic ".github_pat_setup"

Cause: the two internal helpers .github_pat_setup() / .github_pat_run_prefix() are @noRd, so no .Rd is generated; their @description blocks cross-reference each other via [.github_pat_X()] markdown links, which roxygen then can't resolve.

Fix: switch the references from markdown links to plain backticks (`.github_pat_X()`). Prose stays explanatory, no link is attempted, no warning.

Diff

  • R/utils.R:42: [.github_pat_run_prefix()]`.github_pat_run_prefix()`
  • R/utils.R:57: [.github_pat_setup()]`.github_pat_setup()`

No .Rd change (the affected helpers are @noRd), no NAMESPACE change, no behaviour change. devtools::document() is now clean.

Test plan

  • devtools::document(): no warnings.
  • CI green.

`R/utils.R` had two `@description` blocks for the internal
`.github_pat_setup` / `.github_pat_run_prefix` helpers that
cross-referenced each other via `[.github_pat_X()]` markdown links.
Both helpers are `@noRd` so no `.Rd` is generated, and roxygen
emitted "Could not resolve link to topic" warnings on every
`devtools::document()` run. Switched the references to plain
backticks (`.github_pat_X()`): the prose stays explanatory, no link
is attempted, no warning. Pure docstring cleanup; no `.Rd`, no
NAMESPACE, no behaviour change.
@VincentGuyader VincentGuyader merged commit 4c166e2 into master May 13, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant