Skip to content

Add missing TypeInfo::as_set().#23943

Merged
alice-i-cecile merged 3 commits intobevyengine:mainfrom
eswartz:add_dynamic_as_set
Apr 22, 2026
Merged

Add missing TypeInfo::as_set().#23943
alice-i-cecile merged 3 commits intobevyengine:mainfrom
eswartz:add_dynamic_as_set

Conversation

@eswartz
Copy link
Copy Markdown
Contributor

@eswartz eswartz commented Apr 22, 2026

Objective

  • Adds a missing conversion to the bevy::reflect::TypeInfo zoo.

Solution

  • Copied the simple impl and impl_cast_method! invocations used in other types.

Testing

  • Added unit test.

@github-actions
Copy link
Copy Markdown
Contributor

Welcome, new contributor!

Please make sure you've read our contributing guide, as well as our policy regarding AI usage, and we look forward to reviewing your pull request shortly ✨

@kfc35 kfc35 added D-Trivial Nice and easy! A great choice to get started with Bevy A-Reflection Runtime information about types S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Apr 22, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in Reflection Apr 22, 2026
@alice-i-cecile alice-i-cecile 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 Apr 22, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 22, 2026
@alice-i-cecile alice-i-cecile added the C-Bug An unexpected or incorrect behavior label Apr 22, 2026
Merged via the queue into bevyengine:main with commit 6e7b436 Apr 22, 2026
48 checks passed
@github-project-automation github-project-automation Bot moved this from Needs SME Triage to Done in Reflection Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Reflection Runtime information about types C-Bug An unexpected or incorrect behavior D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants