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

(incomplete) update for upcoming ggplot2 version #124

Merged
merged 2 commits into from
Jan 24, 2024
Merged

(incomplete) update for upcoming ggplot2 version #124

merged 2 commits into from
Jan 24, 2024

Conversation

teunbrand
Copy link
Contributor

Hi there,

We have been preparing for a new release of ggplot2 and during a reverse dependency check, it became apparent that the prospective ggplot2 3.5.0 would break afex.

This PR updates some code around legends to be compatible with both the current CRAN version and the 3.5.0 release candidate of ggplot2 simultaneously. Please note that in addition to the changes proposed in this PR, there were some visual changes that you might want to inspect manually, as I lack the domain knowledge to judge their correctness. Moreover, it also seems #119 is recurring with regards to some plots in the vignettes.

To test the code changes with the release candidate, you can install it with the code below:

remotes::install_github("tidyverse/ggplot2", ref = remotes::github_pull("5592"))

The release of ggplot2 3.5.0 is scheduled for the 12th of Februari. The progress of the release can be tracked in tidyverse/ggplot2#5588. We hope that this PR might help afex be ready for future changes.

@singmann
Copy link
Owner

Thanks, really helpful!

@singmann singmann merged commit 1e3ee8d into singmann:master Jan 24, 2024
0 of 5 checks passed
@teunbrand teunbrand deleted the ggplot2_3.5.0 branch January 24, 2024 14:18
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.

2 participants