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

Bevy 0.14 #129

Closed
wants to merge 1 commit into from
Closed

Bevy 0.14 #129

wants to merge 1 commit into from

Conversation

Seldom-SE
Copy link

Just got this working for our project, so we probably won't be updating it further, but feel free to fork bc it needs a bit more work. The tests are failing bc Color::lerp returns a Color::LinearRgba, and the bevy-inspector-egui dev-dependency is commented out

@djeedai
Copy link
Owner

djeedai commented Jul 6, 2024

Thanks @Seldom-SE! I've started a local branch with your change, and I'll append the example fixes once bevy-inspector-egui is available. Note that we don't need ColorLerper anymore; I used the new Mix trait, and that fixes the tests automatically. This has the advantage that if the start and end colors are in the same color space already, then no conversion occurs.

@wakearray
Copy link

I'll append the example fixes once bevy-inspector-egui is available.

bevy-inspector-egui version 0.25.0 supporting Bevy 0.14 is now available 👍

@extrawurst
Copy link

works great for me in my 0.14 project!

@djeedai
Copy link
Owner

djeedai commented Jul 8, 2024

@Seldom-SE I've merged and published, thanks for your PR that helped fast forward quite a few things. I'm sorry I forgot to add the co-authored-by tag in the commit to credit you though 🤦

@Seldom-SE
Copy link
Author

All good! Closing in favor of the new release

@Seldom-SE Seldom-SE closed this Jul 9, 2024
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.

4 participants