Skip to content

Conversation

MrGVSV
Copy link
Member

@MrGVSV MrGVSV commented Feb 3, 2024

Objective

&'static str doesn't implement Reflect. I don't think this was intentionally excluded.

Solution

Make &'static str implement Reflect.


Changelog

  • Implement Reflect and friends for &'static str
  • Add missing Reflect::debug implementation for Cow<'static, str>

@MrGVSV MrGVSV added C-Usability A targeted quality-of-life change that makes Bevy easier to use A-Reflection Runtime information about types labels Feb 3, 2024
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Feb 4, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Feb 4, 2024
Merged via the queue into bevyengine:main with commit 71be08a Feb 4, 2024
tjamaan pushed a commit to tjamaan/bevy that referenced this pull request Feb 6, 2024
# Objective

`&'static str` doesn't implement `Reflect`. I don't think this was
intentionally excluded.

## Solution

Make `&'static str` implement `Reflect`.

---

## Changelog

- Implement `Reflect` and friends for `&'static str`
- Add missing `Reflect::debug` implementation for `Cow<'static, str>`
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-Usability A targeted quality-of-life change that makes Bevy easier to use 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: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants