-
Notifications
You must be signed in to change notification settings - Fork 73
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
fix: ensure compatibility with upcoming ggplot2 v3.4.3 #269
Conversation
Hi there, This is just an update that we've set a release date for the ggplot2 version that includes the guide changes, and it is set to 12 February. If you'd like to test this against the release candidate, you can install it via the code below: remotes::install_github("tidyverse/ggplot2", ref = remotes::github_pull("5592")) The progress of the release can be tracked in tidyverse/ggplot2#5588 |
many thanks @teunbrand I'll have a look soon (hopefully next week) |
This is just a friendly reminder that the release is due soon. |
thanks @teunbrand, I know I know! Don't think I am not very stressed because of that :) But at least you can say to CRAN team you warned us long time ago, this should be enough for the CRAN team to accept your new release. Then we will receive an email saying ggiraph will be removed from cran if we don't fix quickly things |
Aw sorry I didn't mean to stress you out! If you need any input please let me know |
@teunbrand thank you, it should be ok. I think it's just a matter of few hours of work. |
- integrate Pana updates; #269 - add new param `staplewidth`
- integrate Pana's updates; #269 - add new param `staplewidth` - remove copy of dapply - tests: remove failing tests (will fix and bring them back eventually later) - tests: dont use `ggiraphOutput()` - deprecate `ggiraph()`
Compatibility with upcoming ggplot2 v3.4.3: