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

Bump Bevy to 0.15.0-rc.3. #27

Merged
merged 3 commits into from
Nov 11, 2024
Merged

Conversation

andriyDev
Copy link
Contributor

Now we're ready for the 0.15 release!

Some notable weirdness to this bump: Reflect has been split into PartialReflect and Reflect. TL;DR - PartialReflect just means that you can access the fields of the represented type. Reflect on the other hand is an actual instance of the represented type. (or at least something like that). Regardless, ReflectDeserializer now spits out PartialReflect types, which means we need to do more work to extract the correct types.

@Bluefinger Bluefinger self-assigned this Nov 10, 2024
@Bluefinger
Copy link
Owner

I'll double check this against what I've done with #26, but thanks for the PR. If bevy v0.15 goes live before I get round to finishing off that other PR, I'll likely just merge this PR first and rebase the other one on top to unblock folks.

src/plugin.rs Outdated Show resolved Hide resolved
Copy link
Owner

@Bluefinger Bluefinger left a comment

Choose a reason for hiding this comment

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

LGTM, just needs to have that Typed trait constraint moved to be part of SeedableEntropySource and then it is more or less ready to merge.

This trait already holds the majority of trait requirements, so we might as well add it there as well.
Seed512 doesn't impl `Typed` so reflection is broken here because of it.
Copy link
Owner

@Bluefinger Bluefinger left a comment

Choose a reason for hiding this comment

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

Looking good, and good job! Thanks for being patient with some of my feedback 😄

@Bluefinger Bluefinger merged commit 57806e7 into Bluefinger:main Nov 11, 2024
11 checks passed
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