Skip to content

Add Ord impl and Debug case to Severity.#23408

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
AlephCubed:ErrorSeverity
Mar 18, 2026
Merged

Add Ord impl and Debug case to Severity.#23408
alice-i-cecile merged 2 commits intobevyengine:mainfrom
AlephCubed:ErrorSeverity

Conversation

@AlephCubed
Copy link
Contributor

Follow up to suggestions made in #22201.

Adds Ord and PartialOrd derives to Severity, as well as a Debug case. Also some bonus doc fixes.

@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-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it labels Mar 18, 2026
@github-project-automation github-project-automation bot moved this to Needs SME Triage in ECS Mar 18, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Mar 18, 2026
Merged via the queue into bevyengine:main with commit 9c5a65b Mar 18, 2026
45 checks passed
@github-project-automation github-project-automation bot moved this from Needs SME Triage to Done in ECS Mar 18, 2026
@AlephCubed AlephCubed deleted the ErrorSeverity branch March 18, 2026 20:25
github-merge-queue bot pushed a commit that referenced this pull request Mar 20, 2026
Followup to #22201. ~~Includes the changes in #23408, which is currently
in the queue.~~

Adds a new default error handler that defers to an error's `Severity`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants