Skip to content

Enable state scoped entities by default#19354

Merged
alice-i-cecile merged 4 commits intobevyengine:mainfrom
mockersf:enable-state-scoed-entities-by-default
May 26, 2025
Merged

Enable state scoped entities by default#19354
alice-i-cecile merged 4 commits intobevyengine:mainfrom
mockersf:enable-state-scoed-entities-by-default

Conversation

@mockersf
Copy link
Copy Markdown
Member

Objective

  • Enable state scoped entities by default
  • Provide a way to disable it when needed

@mockersf mockersf added the A-States App-level states machines label May 24, 2025
Copy link
Copy Markdown
Contributor

@benfrankel benfrankel left a comment

Choose a reason for hiding this comment

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

Makes sense to me.

Co-authored-by: Ben Frankel <ben.frankel7@gmail.com>
@janhohenheim janhohenheim added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it D-Straightforward Simple bug fixes and API improvements, docs, test and examples labels May 25, 2025
@janhohenheim
Copy link
Copy Markdown
Member

Note: Improves usability of #19296

Copy link
Copy Markdown
Contributor

@bushrat011899 bushrat011899 left a comment

Choose a reason for hiding this comment

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

I think this makes sense. Just an ergonomics question that might be better suited to a follow-up.

@janhohenheim janhohenheim added this to the 0.17 milestone May 25, 2025
@alice-i-cecile alice-i-cecile added this pull request to the merge queue May 26, 2025
Merged via the queue into bevyengine:main with commit 8a223be May 26, 2025
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-States App-level states machines 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants