Skip to content

Bevy curve/shape cleanup - #25651

Merged
alice-i-cecile merged 7 commits into
bevyengine:mainfrom
mockersf:bevy_curve_shape-cleanup
Sep 2, 2026
Merged

Bevy curve/shape cleanup#25651
alice-i-cecile merged 7 commits into
bevyengine:mainfrom
mockersf:bevy_curve_shape-cleanup

Conversation

@mockersf

@mockersf mockersf commented Sep 2, 2026

Copy link
Copy Markdown
Member

Objective

  • the bevy_curve / bevy_shape split (Bevy curve #25380 / Bevy shapes #25302) had a few issues
  • missing licenses files
  • tool generating images didn't update path
  • bevy_animation needs bevy_curve/reflect
  • some alloc gates in bevy_curve seem wrong
  • dependencies are not specified in the same order as other crates

Solution

  • fix them

@mockersf mockersf added this to the 0.20 milestone Sep 2, 2026
@mockersf mockersf added the C-Code-Quality A section of code that is hard to understand or change label Sep 2, 2026
@Zeophlite Zeophlite added D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Sep 2, 2026
@mnmaita mnmaita 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 Sep 2, 2026
@alice-i-cecile
alice-i-cecile added this pull request to the merge queue Sep 2, 2026
Merged via the queue into bevyengine:main with commit 22649b4 Sep 2, 2026
51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@Zeophlite Zeophlite Zeophlite approved these changes
@mnmaita mnmaita mnmaita approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

C-Code-Quality A section of code that is hard to understand or change 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

Milestone

0.20

Development

Successfully merging this pull request may close these issues.

4 participants

@mockersf @Zeophlite @mnmaita @alice-i-cecile