search

extendr / rextendr

An R package that helps scaffolding extendr-enabled packages or compiling Rust code dynamically
https://extendr.github.io/rextendr/
Other
175 stars 25 forks source link

fix: use the CRAN suggesting `SystemRequirements` field #310

Closed eitsupi closed 9 months ago

eitsupi commented 11 months ago

From https://cran.r-project.org/web/packages/using_rust.html

The package should declare SystemRequirements: Cargo (Rust's package manager), rustc

JosiahParry commented 10 months ago

@eitsupi can you use styler::style_package() to fix the linter? @Ilia-Kosenkov can you review this PR? It's simple enough and addresses the requirements in https://cran.r-project.org/web/packages/using_rust.html

eitsupi commented 10 months ago

@eitsupi can you use styler::style_package() to fix the linter?

The error in CI has nothing to do with this change. The part the linter is complaining about has already been fixed in the main branch by #309

Ilia-Kosenkov commented 10 months ago

Do a git merge main to pull in lintr fixes.