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

Upgrade to bevy 0.15 #466

Merged
merged 1 commit into from
Dec 21, 2024
Merged

Upgrade to bevy 0.15 #466

merged 1 commit into from
Dec 21, 2024

Conversation

haihala
Copy link
Contributor

@haihala haihala commented Dec 4, 2024

This is blocked by bevy_ggrs, but I thought I might get the ball rolling somewhat to speed up the overall process. I put temporary deps to my fork of bevy_ggrs, which I used for the upgrade PR.

Like I said in my bevy_ggrs PR, I'm not a 100% certain I got it right. Everything compiles, tests pass, clippy is happy and I got the hello example to work just fine, which gives me a lot of hope, but I'm still not confident. Please review carefully.

Hope this helps with the upgrade.

@johanhelsing johanhelsing added the dependencies Pull requests that update a dependency file label Dec 8, 2024
@johanhelsing
Copy link
Owner

Yeah, I think the wasm build broke because web-sys got upgraded in cargo toml, and it doesn't seem to follow semver... perhaps we should pin the patch version? Won't let it block this PR, though.

PR looks great, thanks a lot! Will merge as soon as bevy_ggrs is out and this depends on the crates release.

@johanhelsing johanhelsing added this to the 0.11 milestone Dec 12, 2024
@johanhelsing
Copy link
Owner

I cherry-picked the web-sys commit to #473 and updated all other dependencies except bevy and bevy_ggrs. Feel free to either rebase or merge main into this one.

@johanhelsing
Copy link
Owner

bevy_ggrs 0.17 is out now, so this is unblocked :)

@haihala
Copy link
Contributor Author

haihala commented Dec 19, 2024

Did some rebasing and got it to build, will rely on the pipe to catch problems.

@johanhelsing johanhelsing merged commit 8f7cba6 into johanhelsing:main Dec 21, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants