Hiroaki Yutani

License about Highcharts

2

I'm not a license expert, but let me quickly share my concern about using Highcharts in an R package. As you probably already notice, the license of Highcharts is a...

list2() doesn't evaluate arguments in way ellipsis::check_dots_used() checks

2

``` r f Error: 1 components of `...` were not used. #> #> We detected these problematic arguments: #> * `a` #> #> Did you misspecify an argument? ```

It seems ggplot2 has no documentation that actually explains how to use `ggplot_add()`. Maybe it should be in [the Extending ggplot2 vignette](https://ggplot2.tidyverse.org/articles/extending-ggplot2.html)? See https://github.com/tidyverse/ggplot2/issues/4983 for an example of confusion.

See https://github.com/extendr/libR-sys/issues/95#issuecomment-1037084923 Currently, libR-sys exports all symbols wildly, including those from the C standard library. I think we should keep the exports minimal to avoid confusion. This pull request tries...

As we found on #95, half of the functions that libR-sys exports are NOT from R (e.g., `lgamma` is from ``). I'm saying "half" because the diff on #96 would...

[Windows] Is it possible to include Rust toolchain in Rtools?

18

Currently libR-sys requires users to install standalone MSYS2 for libclang, but as https://github.com/extendr/extendr/issues/13 suggests, it might be good to use Rtools' MSYS2 for simplicity. Rtools' MSYS uses a dedicated repository,...

As this issue is closed, I need to tackle the issue again... https://github.com/r-lib/pkgbuild/issues/115

Probably users want to set up GitHub Actions CI.

Set CARGO_HOME in Makevars

2

I got this review comment from CRAN: > Checking this creates ~/.cargo sized 82MB, in violation of the CRAN Policy. > Please fix as necessary and resubmit. It seems we...