Skip to content

Declare head/runif imports in graph_svg()#22

Merged
TroyHernandez merged 1 commit into
mainfrom
fix-graph-svg-imports
Apr 24, 2026
Merged

Declare head/runif imports in graph_svg()#22
TroyHernandez merged 1 commit into
mainfrom
fix-graph-svg-imports

Conversation

@TroyHernandez
Copy link
Copy Markdown
Contributor

Summary

  • Adds `@importFrom utils head` and `@importFrom stats runif` to `graph_svg()`.
  • Clears the only remaining R CMD check NOTE: "no visible global function definition for 'head' / 'runif'".

Test plan

  • `tinypkgr::check()` locally — Status: OK, 0 notes.
  • CI passes on ubuntu + macos with no NOTE.

R CMD check --as-cran flagged graph_svg() for calling head() and runif()
without declaring them, producing a NOTE. Added @importFrom roxygen
tags so NAMESPACE imports them explicitly.
@TroyHernandez TroyHernandez merged commit 6eb04d6 into main Apr 24, 2026
4 checks passed
@TroyHernandez TroyHernandez deleted the fix-graph-svg-imports branch April 24, 2026 21:02
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