Skip to content

Allow #[derive(Bundle)] on tuple structs (take 2)#3634

Closed
dataphract wants to merge 1 commit intobevyengine:mainfrom
dataphract:feat/bundle-tuple-struct
Closed

Allow #[derive(Bundle)] on tuple structs (take 2)#3634
dataphract wants to merge 1 commit intobevyengine:mainfrom
dataphract:feat/bundle-tuple-struct

Conversation

@dataphract
Copy link
Member

This is a rebase of @Veykril's work in #2499 onto the latest main. Fixes #3537.

@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Jan 10, 2022
@james7132 james7132 added A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible C-Usability A targeted quality-of-life change that makes Bevy easier to use and removed S-Needs-Triage This issue needs to be labelled labels Jan 10, 2022
Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

Looks good! Some more comments on this derive macro would help with maintainability in the future, but this PR doesn't make the existing situation any worse (and adds nice tests).

Co-authored-by: dataphract <dataphract@gmail.com>
@mockersf
Copy link
Member

So... it seems this PR was missed but is still needed... @dataphract are you ready for take 3?

@superdump superdump added the S-Adopt-Me The original PR author has no intent to complete this work. Pick me up! label Jan 31, 2023
@SET001
Copy link
Member

SET001 commented Nov 24, 2023

fixed with 10561

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible C-Usability A targeted quality-of-life change that makes Bevy easier to use S-Adopt-Me The original PR author has no intent to complete this work. Pick me up!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make #[derive(Bundle)] work on tuple structs

7 participants