Hi there,
We've been preparing a new major release for ggplot2 and found an issue during a reverse dependency check.
One example in your package fails in the line below:
|
#' df <- h %>% plot_CV() %$% data |
While I don't have much experience with magrittr::%$%(), AFAICT the code assumes that the plot object can be seen as some type of environment, which no longer holds. I presume because we migrated from S3 to S7 and a plot object is no longer a list.
You can test your code with the development version of ggplot2 by installing it as follows:
# install.packages("pak")
pak::pak("tidyverse/ggplot2")We aim to release the new ggplot2 version in about 2 weeks, and hope you can submit a fix to CRAN around that time or let us know when ggplot2 should change. Hopefully this will inform you in a timely manner.
Best wishes,
Teun
Hi there,
We've been preparing a new major release for ggplot2 and found an issue during a reverse dependency check.
One example in your package fails in the line below:
Momocs/R/gr-LDA.R
Line 319 in fca92dc
While I don't have much experience with
magrittr::%$%(), AFAICT the code assumes that the plot object can be seen as some type of environment, which no longer holds. I presume because we migrated from S3 to S7 and a plot object is no longer a list.You can test your code with the development version of ggplot2 by installing it as follows:
We aim to release the new ggplot2 version in about 2 weeks, and hope you can submit a fix to CRAN around that time or let us know when ggplot2 should change. Hopefully this will inform you in a timely manner.
Best wishes,
Teun