Skip to content

Add register_resource_with_descriptor#15501

Merged
alice-i-cecile merged 4 commits intobevyengine:mainfrom
chompaa:register_resource_with_descriptor
Sep 30, 2024
Merged

Add register_resource_with_descriptor#15501
alice-i-cecile merged 4 commits intobevyengine:mainfrom
chompaa:register_resource_with_descriptor

Conversation

@chompaa
Copy link
Copy Markdown
Member

@chompaa chompaa commented Sep 28, 2024

Objective

Solution

  • Add World::register_resource_with_descriptor and Components::register_resource_with_descriptor.

Testing

  • Added a test dynamic_resource.

@chompaa chompaa added C-Feature A new feature, making something new possible A-ECS Entities, components, systems, and events D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Sep 28, 2024
@alice-i-cecile alice-i-cecile added the X-Uncontroversial This work is generally agreed upon label Sep 28, 2024
Copy link
Copy Markdown
Contributor

@MichalGniadek MichalGniadek left a comment

Choose a reason for hiding this comment

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

Would it make sense to also use it in the custom_resource_with_layout test?

@alice-i-cecile alice-i-cecile modified the milestone: 0.15 Sep 28, 2024
@chompaa
Copy link
Copy Markdown
Member Author

chompaa commented Sep 28, 2024

Would it make sense to also use it in the custom_resource_with_layout test?

I had this originally, but decided to keep it as is. After mulling it over though I think I'm in agreement.

@pablo-lua pablo-lua added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Sep 29, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Sep 30, 2024
Merged via the queue into bevyengine:main with commit 0d2eb3d Sep 30, 2024
robtfm pushed a commit to robtfm/bevy that referenced this pull request Oct 4, 2024
# Objective

- Fixes bevyengine#15448.

## Solution

- Add `World::register_resource_with_descriptor` and
`Components::register_resource_with_descriptor`.

## Testing

- Added a test `dynamic_resource`.
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 D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it X-Uncontroversial This work is generally agreed upon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add init_resource_with_descriptor

5 participants