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.

Sign up for GitHub

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

Jump to bottom

Adoption of JoJoJet/bevy-trait-query #15970

Open
Inspirateur opened this issue Oct 17, 2024 · 1 comment
Open

Adoption of JoJoJet/bevy-trait-query #15970

Inspirateur opened this issue Oct 17, 2024 · 1 comment
Labels
A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible S-Needs-SME Decision or review from an SME is required X-Controversial There is active debate or serious implications around merging this PR

Comments

@Inspirateur
Copy link

Trait queries are missing in bevy and only covered by https://github.com/JoJoJet/bevy-trait-query/
Upstreaming it would be nicer for bevy users as it is a really common need and 3rd party lib can easily become outdated🥺

(Got told by @alice-i-cecile to open this :p)
@Inspirateur Inspirateur added C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled labels Oct 17, 2024
@Inspirateur Inspirateur mentioned this issue Oct 17, 2024
Closed
@alice-i-cecile alice-i-cecile added A-ECS Entities, components, systems, and events X-Controversial There is active debate or serious implications around merging this PR S-Needs-SME Decision or review from an SME is required and removed S-Needs-Triage This issue needs to be labelled labels Oct 17, 2024
@Inspirateur Inspirateur mentioned this issue Oct 18, 2024
Closed
@giocri
Copy link

giocri commented Oct 20, 2024

yeah this would be immensely useful for my game to i have several types of objects who have a common trait to write to a buffer and would have to do so in the right order which is an absolute pain to do without being able to do them all in the same system

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible S-Needs-SME Decision or review from an SME is required X-Controversial There is active debate or serious implications around merging this PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@alice-i-cecile @Inspirateur @giocri