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 #41

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dsgallups
Copy link

@dsgallups dsgallups commented Dec 6, 2024

Hello! I'd like to use your crate in 0.15. There are a few snags upgrading detailed below. Not sure how you'd like me to handle these. In general though, the upgrade works!

I'm not sure how you would like to proceed on the piano.rs example. I think this example is possible without this dependency.

Additionally, would it be alright to lock dependency versioning for this crate (remove Cargo.lock from .gitignore)? I would like to ensure consistent behavior for this crate since bevy is a tumultuous ecosystem.

There is a bug in Bevy 0.15 where the bevy_gtlf feature will fail compilation: bevyengine/bevy#16671.

In the meantime, dev-dependencies enables all features.
Mysteriously, the input ports have disappeared? This should be investigated
…ing)

bevy_mod_picking should be upgraded to 0.15: aevyrie/bevy_mod_picking#375

Unsure how to proceed, could remove this dependency for this example to compile.
@BlackPhlox
Copy link
Owner

Huge kudos for doing this! Although I won’t be able to review it today, let me know if you need anything else from me in lieu of this and #43.

@dsgallups
Copy link
Author

I've made an analogous example to resolve the piano example. Happy to follow suit in this PR

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