-
Notifications
You must be signed in to change notification settings - Fork 32
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
Update to egui 0.29.0
& Enhancements
#37
Conversation
Just noticed |
Awesome, thank you 🙏 Yeah I need to do a final clean up / documentation pass before I publish egui_flex, I plan to publish it next week. If you want to clean it up beforehand, that would be appreciated! |
i can try :-) What about linting? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thank you so much!
Since this PR is arbitrarily named 'enhancements', would adding #36 to this pr be possible? |
I'd prefere a separate one since this one is already reeeeally big |
Ooops xD |
Now everything should be ready to merge :-) https://github.com/bircni/hello_egui/actions/runs/11087433518 |
Maybe we could also set all versions on the same version number as egui is. So it has the same number and all crate versions here can be managed by the main cargo.toml file with the workspace. This could be done in a seperate PR maybe... - tell me if you want that |
I actually thought about that before but decided against it because it would mean I can't release a breaking change to a crate until the next egui release. For now I'm fine with separate versions (cargo release makes this really easy to manage) |
Do you still need something from me or is it done then?😅 |
It's done, I was just waiting on the pipeline 😄 |
No description provided.