Skip to content

Provide Set Enum for scene_spawner systems#22465

Merged
mockersf merged 3 commits intobevyengine:mainfrom
valenotary:fix/22325
Jan 11, 2026
Merged

Provide Set Enum for scene_spawner systems#22465
mockersf merged 3 commits intobevyengine:mainfrom
valenotary:fix/22325

Conversation

@valenotary
Copy link
Copy Markdown
Contributor

Objective

Fixes #22325

Solution

Copy minimal example from TransformSystem and apply it to minimal necessary source in scene_spawner.rs. Basically, created a new label and added the respective systems in it.

Testing

Didn't think I needed to test it since it's p small and I didn't see any tests specifically revolving around TransformSystem's label, but happy to do so if reviewers deem it necessary (first time trying to contribute in a long time).

Implement a solution for bevyengine#22325

Copy minimal example from `TransformSystem` and apply it to minimal necessary source in `scene_spawner.rs`. Basically, created a new label and added the respective systems in it.
@mrchantey mrchantey self-requested a review January 10, 2026 03:47
Copy link
Copy Markdown
Contributor

@mrchantey mrchantey left a comment

Choose a reason for hiding this comment

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

Simple, well documented, fixes an undestandable issue, lgtm!

@kfc35 kfc35 requested a review from Shatur January 10, 2026 18:35
@kfc35 kfc35 added C-Feature A new feature, making something new possible 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 A-Scenes Composing and serializing ECS objects labels Jan 10, 2026
Copy link
Copy Markdown
Contributor

@Shatur Shatur left a comment

Choose a reason for hiding this comment

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

Thank you! Minor docs suggestion.

Comment thread crates/bevy_scene/src/scene_spawner.rs Outdated
@Shatur Shatur added this to the 0.18 milestone Jan 10, 2026
@Shatur
Copy link
Copy Markdown
Contributor

Shatur commented Jan 10, 2026

Trivial enough to include in 0.18.

valenotary and others added 2 commits January 10, 2026 21:14
Co-authored-by: Hennadii Chernyshchyk <genaloner@gmail.com>
@mockersf mockersf added this pull request to the merge queue Jan 11, 2026
Merged via the queue into bevyengine:main with commit b7fbe57 Jan 11, 2026
38 checks passed
alice-i-cecile pushed a commit that referenced this pull request Jan 13, 2026
# Objective

Fixes #22325

## Solution

Copy minimal example from `TransformSystem` and apply it to minimal
necessary source in `scene_spawner.rs`. Basically, created a new label
and added the respective systems in it.

## Testing

Didn't think I needed to test it since it's p small and I didn't see any
tests specifically revolving around `TransformSystem`'s label, but happy
to do so if reviewers deem it necessary (first time trying to contribute
in a long time).

---------

Co-authored-by: Hennadii Chernyshchyk <genaloner@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Scenes Composing and serializing ECS objects C-Feature A new feature, making something new possible 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

None yet

Development

Successfully merging this pull request may close these issues.

Add labels for scene_spawner_system and scene_spawner

5 participants