-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Implement Clone for VisibilityBundle and SpatialBundle #10394
Implement Clone for VisibilityBundle and SpatialBundle #10394
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you check if SpatialBundle (and ideally our other premade bundle types) need this treatment too?
Good call, I Implemented |
Huh, why did it fail building wasm? |
Looks like a semver violating change / new bug in one of our dependencies :( We may need to pin async_executor: smol-rs/async-executor#72 (thanks @Bluefinger). |
) # Objective Had an issue where I had `VisibilityBundle` inside a bundle that implements `Clone`, but since `VisibilityBundle` doesn't implement `Clone` that wasn't possible. This PR fixes that. ## Solution Implement `Clone` for `VisibilityBundle` by deriving it. And also `SpatialBundle` too because why not. --- ## Changelog - Added implementation for `Clone` on `VisibilityBundle` and `SpatialBundle`.
Objective
Had an issue where I had
VisibilityBundle
inside a bundle that implementsClone
, but sinceVisibilityBundle
doesn't implementClone
that wasn't possible. This PR fixes that.Solution
Implement
Clone
forVisibilityBundle
by deriving it. And alsoSpatialBundle
too because why not.Changelog
Clone
onVisibilityBundle
andSpatialBundle
.