Skip to content

fix: replace non-API Rf_findVar with R_getVar/R_getVarEx - #133

Merged
aviezerl merged 4 commits into
masterfrom
fix/cran-non-api-calls
Mar 19, 2026
Merged

fix: replace non-API Rf_findVar with R_getVar/R_getVarEx#133
aviezerl merged 4 commits into
masterfrom
fix/cran-non-api-calls

Conversation

@aviezerl

@aviezerl aviezerl commented Mar 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • Replaced 8 Rf_findVar calls with R_getVar/R_getVarEx across 5 C++ files
  • Added backward-compatibility shim for R < 4.5.0 in src/naryn.h
  • CRAN compliance fix for R 4.6.0 (deadline: April 9, 2026)

Test plan

  • CI R CMD check passes
  • Package compiles successfully

aviezerl added 4 commits March 19, 2026 17:47
@aviezerl
fix: replace non-API Rf_findVar with R_getVar/R_getVarEx for R 4.6.0 …
1dc5f07 
…compatibility
@aviezerl
fix: cast TRUE to (Rboolean)TRUE for C++ compatibility with R_getVar/…
453a108 
…R_getVarEx

Also restore portability note in cran-comments.md.
@aviezerl
fix: resolve CRAN pre-test warning for pipe.Rd
cb70687 
Removed @Usage, @param, and @return tags from the %>% re-export
roxygen block. The \usage{} section in the generated Rd file caused
R CMD check to warn about a function documented but not defined in
package code. Bump version to 2.6.34.
@aviezerl
fix: add .Rd2pdf temp dirs to .Rbuildignore
0db1260
@aviezerl
aviezerl merged commit 104521c into master Mar 19, 2026
4 checks passed
@aviezerl
aviezerl deleted the fix/cran-non-api-calls branch March 19, 2026 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@aviezerl