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.

Sign up for GitHub

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

Jump to bottom

Convert to S7 #380

Open
thomasp85 opened this issue Aug 27, 2024 · 1 comment
Open

Convert to S7 #380

thomasp85 opened this issue Aug 27, 2024 · 1 comment
Labels
feature a feature request or enhancement

Comments

@thomasp85
Copy link
Owner

There are a bunch of user grievances from the dispatch constraints of S3. This makes patchwork a great candidate for kicking the tyres of S7
@thomasp85 thomasp85 added the feature a feature request or enhancement label Aug 27, 2024
@thomasp85
Copy link
Owner Author

Initial tests reveal that the double dispatch works, but only by overwriting the + operator (something patchwork shouldn't do)

Since patchwork doesn't own the classes it operates on it has to wait for ggplot2 to update to S7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature a feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thomasp85