Skip to content

Bump egui + bevy versions - #2820

Open
oleflb wants to merge 3 commits into
HULKs:mainfrom
oleflb:bump/egui-bevy
Open

Bump egui + bevy versions#2820
oleflb wants to merge 3 commits into
HULKs:mainfrom
oleflb:bump/egui-bevy

Conversation

@oleflb

@oleflb oleflb commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Why? What?

Bumps bevy to 0.19.0 and egui to 0.35.0. This also requires to increase MSRV to 1.95.0
Motivation is the new simulator, which will need features from bevy 0.19

  • Fixes #

ToDo / Known Issues

If this is a WIP describe which problems are to be fixed.

Ideas for Next Iterations (Not This PR)

If there are some improvements that could be done in a next iteration, describe them here.

How to Test

  • Check apps still behave as expected

@github-project-automation github-project-automation Bot moved this to In Progress in Development Jul 18, 2026
@oleflb oleflb moved this from In Progress to Review in Development Jul 18, 2026
@knoellle knoellle self-assigned this Jul 24, 2026

@knoellle knoellle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor comment, LGTM otherwise

Comment on lines 10 to 13
ctx.on_begin_pass("keybinds", Arc::new(|ui| begin_frame(ui.ctx())))
}

fn begin_frame(ctx: &Context) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This function isn't used anywhere else. Why not change its signature instead of wrapping it in a closure?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

2 participants