Skip to content

add methods to *EntityMut to UnsafeEntityCell as an escape hatch#21234

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
hymm:expose-more-unsafe-entity-cell
Sep 26, 2025
Merged

add methods to *EntityMut to UnsafeEntityCell as an escape hatch#21234
alice-i-cecile merged 1 commit intobevyengine:mainfrom
hymm:expose-more-unsafe-entity-cell

Conversation

@hymm
Copy link
Contributor

@hymm hymm commented Sep 26, 2025

Objective

  • Sometimes you want to break mutability rules to get multiple mutable components off of an entity.

Solution

Testing

  • haven't really tested, but changes are straightforward

@alice-i-cecile
Copy link
Member

alice-i-cecile commented Sep 26, 2025

Related to #21182.

alice-i-cecile
alice-i-cecile approved these changes Sep 26, 2025
View reviewed changes
Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

Very straightforward.

@alice-i-cecile alice-i-cecile added A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Sep 26, 2025
@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 Sep 26, 2025
@alice-i-cecile alice-i-cecile modified the milestones: 0.17, 0.18 Sep 26, 2025
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Sep 26, 2025
Merged via the queue into bevyengine:main with commit 1032559 Sep 26, 2025
42 checks passed
@hymm hymm deleted the expose-more-unsafe-entity-cell branch October 1, 2025 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alice-i-cecile alice-i-cecile alice-i-cecile approved these changes

@chescock chescock Awaiting requested review from chescock

+1 more reviewer

@cBournhonesque cBournhonesque cBournhonesque approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A-ECS Entities, components, systems, and events 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

None yet

Milestone

0.18

Development

Successfully merging this pull request may close these issues.

3 participants

@hymm @alice-i-cecile @cBournhonesque