Skip to content

Roll back updates to update_geom/stat_defaults() #5837

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

Merged
merged 2 commits into from
Apr 18, 2024

Conversation

teunbrand
Copy link
Collaborator

This PR reverts #5781 in the prep/3.5.1 branch.

Briefly, #5781 was giving some grief with reverse dependencies so this PR allows us to postpone #5781 to a later release.

@teunbrand teunbrand requested a review from thomasp85 April 12, 2024 07:22
@teunbrand teunbrand added this to the ggplot2 3.5.1 milestone Apr 12, 2024
@teunbrand teunbrand changed the title Revert #5781 Roll back updates to update_geom/stat_defaults() Apr 12, 2024
@thomasp85
Copy link
Member

We need this to be a PR against main with the new release structure

@teunbrand teunbrand changed the base branch from prep/3.5.1 to main April 15, 2024 08:16
@teunbrand teunbrand changed the base branch from main to prep/3.5.1 April 15, 2024 08:16
@teunbrand teunbrand changed the base branch from prep/3.5.1 to main April 15, 2024 08:29
@teunbrand
Copy link
Collaborator Author

Thanks for that pointer! I used git rebase --onto to replay the commits on the main branch.

Copy link
Member

@thomasp85 thomasp85 left a comment

Choose a reason for hiding this comment

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

LGTM - can you revive the old PR so we keep a tab on it for after the patch release

@teunbrand
Copy link
Collaborator Author

teunbrand commented Apr 18, 2024

I revived the relevant issue #4993. I think I cannot make a new PR from main until the rollback is complete.

@teunbrand teunbrand merged commit be4b7b8 into tidyverse:main Apr 18, 2024
@teunbrand teunbrand deleted the revert_updaters branch April 18, 2024 07:37
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