Skip to content

Expose bevy math ops#14863

Merged
alice-i-cecile merged 5 commits intobevyengine:mainfrom
Lubba-64:expose-bevy-math-ops
Aug 22, 2024
Merged

Expose bevy math ops#14863
alice-i-cecile merged 5 commits intobevyengine:mainfrom
Lubba-64:expose-bevy-math-ops

Conversation

@Lubba-64
Copy link
Copy Markdown
Contributor

Objective

Solution

  • Copy docs for wrapper methods, make sure they are consistent with the original docs except for the section on precision.

@IQuick143 IQuick143 added A-Math Fundamental domain-agnostic mathematical operations S-Needs-Review Needs reviewer attention (from anyone!) to move forward C-Docs An addition or correction to our documentation D-Straightforward Simple bug fixes and API improvements, docs, test and examples C-Usability A targeted quality-of-life change that makes Bevy easier to use labels Aug 22, 2024
@alice-i-cecile alice-i-cecile added M-Release-Note Work that should be called out in the blog due to impact 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 Aug 22, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Aug 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 22, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Aug 22, 2024
github-merge-queue bot pushed a commit that referenced this pull request Aug 22, 2024
# Objective

- Fixes #14796 

## Solution

- Copy docs for wrapper methods, make sure they are consistent with the
original docs except for the section on precision.
Merged via the queue into bevyengine:main with commit b922896 Aug 22, 2024
@Lubba-64 Lubba-64 deleted the expose-bevy-math-ops branch August 24, 2024 16:01
@alice-i-cecile
Copy link
Copy Markdown
Member

Thank you to everyone involved with the authoring or reviewing of this PR! This work is relatively important and needs release notes! Head over to bevyengine/bevy-website#1676 if you'd like to help out.

@alice-i-cecile alice-i-cecile removed the M-Release-Note Work that should be called out in the blog due to impact label Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Math Fundamental domain-agnostic mathematical operations C-Docs An addition or correction to our documentation C-Usability A targeted quality-of-life change that makes Bevy easier to use 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

Development

Successfully merging this pull request may close these issues.

Follow-up: Make bevy_math's ops public (and document the methods)

4 participants