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

Update to Bevy 0.12 #439

Merged
merged 1 commit into from
Nov 8, 2023
Merged

Update to Bevy 0.12 #439

merged 1 commit into from
Nov 8, 2023

Conversation

tim-blackbird
Copy link
Contributor

@tim-blackbird tim-blackbird commented Nov 1, 2023

Just waiting on the Bevy 0.12 release to move out of draft mode :)

Comment on lines -91 to +94
systems::init_async_scene_colliders.after(bevy::scene::scene_spawner_system),
systems::init_async_scene_colliders,
Copy link
Contributor Author

@tim-blackbird tim-blackbird Nov 1, 2023

Choose a reason for hiding this comment

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

scene_spawner_system runs in a schedule before PostUpdate

Comment on lines +1638 to +1639
app.add_plugins(HeadlessRenderPlugin)
.add_systems(PostUpdate, init_async_scene_colliders);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Old test was failing because the scene_spawner_system was moved to its own schedule that runs in between Update and PostUpdate.

Updated to match the way init_async_scene_colliders is added by the actual plugin.

@thombruce thombruce mentioned this pull request Nov 4, 2023
10 tasks
@rdrpenguin04
Copy link

Bevy 0.12 is officially out

Copy link

@mnmaita mnmaita left a comment

Choose a reason for hiding this comment

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

Amazing work, you are always a step ahead of releases! 😄 I just tested this branch in my project and it works well so far.

bevy_rapier2d/Cargo.toml Outdated Show resolved Hide resolved
bevy_rapier3d/Cargo.toml Outdated Show resolved Hide resolved
@TristanCacqueray
Copy link

Should this update the bevy_rapier version to 0.23 ?

@tim-blackbird
Copy link
Contributor Author

Should this update the bevy_rapier version to 0.23 ?

The version should be bumped before this change is released, but I'll leave that up to @sebcrozet.

@sebcrozet sebcrozet merged commit cd78577 into dimforge:master Nov 8, 2023
3 checks passed
@sebcrozet
Copy link
Member

Thanks @devil-ira and @Aceeri for the review!

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.

7 participants