Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

extendr-api 0.7.0 for pola-rs/r-polars#1339 #5

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

eitsupi
Copy link
Member

@eitsupi eitsupi commented Mar 13, 2025

No description provided.

CGMossa and others added 7 commits March 13, 2025 15:10
…r#742)

* replaces `SEXPTYPE` and `Rboolean` to the
new enum-versions from latest `libR-sys`.

* Cargo.toml: Update version of `libR-sys`, otherwise it doesn't work

* build.rs: Added a feature for using `OBJSXP` vs `S4SXP`
The cfg-feature might be excessive, but handling S4 object will
have to depend on R versions eventually, so we are starting
this branching now...

* MSRV: Updated to 1.70 due to a change
in bindgen, which uses `which`, and which
uses `home` which silently updated
their msrv to 1.70.
* Cargo.toml: update libR-sys to latest commit

* added more `non-api` cfg guards

* moved doctests to their own module

* wp: another

* wp: another one

* this test is disabled, yet it still compiles..

* typo

* removed more tests

* update changelog

* update note on CRAN

* `cargo extendr fmt` [skip ci]

---------

Co-authored-by: Josiah Parry <[email protected]>
* Added a `SendSEXP` instead of conversion to
usize

* `cargo fmt`.

* remove unnecessary Deref/DerefMut impl

* add comments to ownership

* typos

* `cargo fmt`

* comments

* remove set_inner() method from SendSEXP

* rename sexp_usize to send_sexp to make the type:

---------

Co-authored-by: Josiah Parry <[email protected]>
* xtask: add helpful message when missing dependencies for `r-cmd-check`

* Makevars: color is auto, not always

* README: add instruction to use developer extendr in rextendr.

* extendr-api: added `non-api` flag

* ci: temporary hack to get past this circular testing BS

* Cargo.toml: we will no longer rely on `patch.crates-io`.

* ci: added ` REXTENDR_SKIP_DEV_TESTS: TRUE`
* xtask: add helpful message when missing dependencies for `r-cmd-check`

* Makevars: color is auto, not always

* README: add instruction to use developer extendr in rextendr.

* extendr-api: added `non-api` flag

* ci: temporary hack to get past this circular testing BS

* Cargo.toml: we will no longer rely on `patch.crates-io`.

* ci: added ` REXTENDR_SKIP_DEV_TESTS: TRUE`
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.

3 participants