Skip to content

Removed invalid ECS archetype constants#21741

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
ElliottjPierce:remove-invalid-locations
Nov 4, 2025
Merged

Removed invalid ECS archetype constants#21741
alice-i-cecile merged 1 commit intobevyengine:mainfrom
ElliottjPierce:remove-invalid-locations

Conversation

@ElliottjPierce
Copy link
Contributor

Objective

This is a tiny clean up to #19451 that removes some now completely unneeded public constants.
I mean, they aren't wrong, but there's no point to their existence.

Solution

Removed ArchetypeRow::INVALID and ArchetypeId::INVALID and extend migration guide.

@ElliottjPierce ElliottjPierce added D-Trivial Nice and easy! A great choice to get started with Bevy A-ECS Entities, components, systems, and events C-Code-Quality A section of code that is hard to understand or change S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Nov 4, 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 Nov 4, 2025
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Nov 4, 2025
Merged via the queue into bevyengine:main with commit 8919ad1 Nov 4, 2025
47 checks passed
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-Code-Quality A section of code that is hard to understand or change 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.

2 participants