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

use shading enums to remove warnings #65

Merged
merged 3 commits into from
Dec 13, 2023
Merged

use shading enums to remove warnings #65

merged 3 commits into from
Dec 13, 2023

Conversation

rafaqz
Copy link
Collaborator

@rafaqz rafaqz commented Dec 12, 2023

@simon was this enum available in Makie 0.19 or should I fix the version to 0.20 ?

@SimonDanisch
Copy link
Member

This is new as of 0.20!

@rafaqz
Copy link
Collaborator Author

rafaqz commented Dec 12, 2023

Ok thanks. Do you think its worth trying to support both 0.19 and 0.20 here, or just switching to 0.20?

@SimonDanisch
Copy link
Member

I've been in the force updating camp most of the time to optimize maintainer time ;)

@rafaqz
Copy link
Collaborator Author

rafaqz commented Dec 12, 2023

Me to but I get in trouble for it 😅

@rafaqz
Copy link
Collaborator Author

rafaqz commented Dec 12, 2023

Haha in the end I also have to downgrade everything to 0.19 because MakieDraw is totally broken on 0.20

@rafaqz rafaqz merged commit 807df4d into master Dec 13, 2023
2 checks passed
@rafaqz rafaqz deleted the shading branch December 13, 2023 12:36
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