Skip to content

[Merged by Bors] - Update Box vertices comment #7055

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

[Merged by Bors] - Update Box vertices comment #7055

wants to merge 1 commit into from

Conversation

foxzool
Copy link
Contributor

@foxzool foxzool commented Dec 29, 2022

Old comment is Z-up , Fix comment for bevy Y-up

Objective

  • Update Box vertices comment for bevy Y-up

Solution

  • Update comment for Y-up

Changelog

None

Migration Guide

None

Old comment is Z-up ,  Fix docs for bevy Y-up
@nicopap nicopap added the C-Code-Quality A section of code that is hard to understand or change label Dec 29, 2022
@alice-i-cecile alice-i-cecile added C-Docs An addition or correction to our documentation A-Rendering Drawing game state to the screen labels Dec 29, 2022
@alice-i-cecile
Copy link
Member

bors r+

@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Dec 29, 2022
bors bot pushed a commit that referenced this pull request Dec 29, 2022
Old comment is Z-up ,  Fix comment for bevy Y-up

# Objective

- Update Box vertices comment for bevy Y-up

## Solution

- Update comment for Y-up

---

## Changelog

None

## Migration Guide

None
@bors bors bot changed the title Update Box vertices comment [Merged by Bors] - Update Box vertices comment Dec 29, 2022
@bors bors bot closed this Dec 29, 2022
alradish pushed a commit to alradish/bevy that referenced this pull request Jan 22, 2023
Old comment is Z-up ,  Fix comment for bevy Y-up

# Objective

- Update Box vertices comment for bevy Y-up

## Solution

- Update comment for Y-up

---

## Changelog

None

## Migration Guide

None
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
Old comment is Z-up ,  Fix comment for bevy Y-up

# Objective

- Update Box vertices comment for bevy Y-up

## Solution

- Update comment for Y-up

---

## Changelog

None

## Migration Guide

None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rendering Drawing game state to the screen C-Code-Quality A section of code that is hard to understand or change C-Docs An addition or correction to our documentation 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.

3 participants