Skip to content

Simplify ambiguity_detection test to use per-app schedule configuration. - #26046

Merged
alice-i-cecile merged 1 commit into
bevyengine:mainfrom
andriyDev:simple-schedule-config
Oct 7, 2026
Merged

Simplify ambiguity_detection test to use per-app schedule configuration.#26046
alice-i-cecile merged 1 commit into
bevyengine:mainfrom
andriyDev:simple-schedule-config

Conversation

@andriyDev

@andriyDev andriyDev commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Objective

  • This code is way more complicated than it needs to be!

Solution

  • Simplify the ambiguity detection test to just use the existing App::configure_schedules function.

Testing

  • None

@andriyDev andriyDev added D-Trivial Nice and easy! A great choice to get started with Bevy A-ECS Entities, components, systems, and events C-Testing A change that impacts how we test Bevy or how users test their apps S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 7, 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 Oct 7, 2026
@alice-i-cecile
alice-i-cecile added this pull request to the merge queue Oct 7, 2026
Merged via the queue into bevyengine:main with commit 5cd69c1 Oct 7, 2026
58 checks passed
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

Assignees

No one assigned

Labels

A-ECS Entities, components, systems, and events C-Testing A change that impacts how we test Bevy or how users test their apps 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

ECS
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andriyDev @alice-i-cecile