Skip to content

Implement accessors#1900

Closed
TheRawMeatball wants to merge 2 commits intobevyengine:mainfrom
TheRawMeatball:accessors
Closed

Implement accessors#1900
TheRawMeatball wants to merge 2 commits intobevyengine:mainfrom
TheRawMeatball:accessors

Conversation

@TheRawMeatball
Copy link
Member

This PR superceeds the monolithic #1393, and implements the core accessor api.

@alice-i-cecile alice-i-cecile added core A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible labels Apr 12, 2021
@mockersf
Copy link
Member

Very neat 👍
Could do with an example though

@alice-i-cecile alice-i-cecile added the S-Needs-Design-Doc This issue or PR is particularly complex, and needs an approved design doc before it can be merged label Apr 23, 2021
@cart cart added the S-Pre-Relicense This PR was made before Bevy added the Apache license. Cannot be merged or used for other work label Jul 23, 2021
@mockersf mockersf removed the S-Pre-Relicense This PR was made before Bevy added the Apache license. Cannot be merged or used for other work label Jul 24, 2021
@alice-i-cecile alice-i-cecile removed the S-Needs-Design-Doc This issue or PR is particularly complex, and needs an approved design doc before it can be merged label Nov 17, 2021
@alice-i-cecile
Copy link
Member

@TheRawMeatball, do you still think this is useful? If so, could you add an example and update the PR description to be more helpful when you get a chance?

@alice-i-cecile alice-i-cecile added A-Tasks Tools for parallel and async work and removed A-Core labels Apr 25, 2022
@TheRawMeatball
Copy link
Member Author

It's actually probably not tbh, i'll just close the PR unless i need it, at which point i'd probably just reimplement

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 A-Tasks Tools for parallel and async work C-Feature A new feature, making something new possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants