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

Changes in upcoming ggplot2 version #153

Merged
merged 5 commits into from
Feb 8, 2024
Merged

Changes in upcoming ggplot2 version #153

merged 5 commits into from
Feb 8, 2024

Conversation

teunbrand
Copy link
Contributor

Hi there,

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

As a brief summary, we've modified the scale's call field to hold a more informative call for the use in error messages and warnings. Unfortunately, this means that the scale equality assumptions in ggthemes no longer hold. This PR does two things:

  • It updates the tests to check for scale equality that doesn't depend on the call field.
  • It pre-emptively replaces recently deprecated theme arguments. This isn't strictly necessary as there is a fallback, but it reduces the warnings a bit.

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 February. The progress of the release can be tracked in tidyverse/ggplot2#5588. We hope that this PR might help ggthemes get out a fix if necessary.

@teunbrand
Copy link
Contributor Author

This is just a kind reminder that the release is scheduled soon.

@jrnold
Copy link
Owner

jrnold commented Feb 8, 2024

Thanks! I'll get this merged.

Do these changes require updating the minimum version of ggplot2 in the dependencies?

Copy link
Owner

@jrnold jrnold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you rebase against main and update NEWS.md with a description of the changes?

@teunbrand
Copy link
Contributor Author

Do these changes require updating the minimum version of ggplot2 in the dependencies?

They do not, it should be forwards and backwards compatible.

@jrnold jrnold merged commit 2cd9393 into jrnold:main Feb 8, 2024
6 of 7 checks passed
@jrnold
Copy link
Owner

jrnold commented Feb 8, 2024

Thanks! Merged. Today, I'll be fixing up misc issues that have arisen over time. I'll submit a new version to CRAN by eod pacific time.

@jrnold
Copy link
Owner

jrnold commented Feb 14, 2024

By the way - I'm looking for someone to take over as maintainer of ggthemes. Do you know anyone?

@teunbrand
Copy link
Contributor Author

I can ask around in a group of fellow ggplot2 enthusiasts, but given that is only a handful of people. My advise would be to ask around on social media and then check if a candidate has prior experience maintaining a ggplot related package.

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