Skip to content

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

Closed
foxzool wants to merge 1 commit intobevyengine:mainfrom
foxzool:master
Closed

[Merged by Bors] - Update Box vertices comment#7055
foxzool wants to merge 1 commit intobevyengine:mainfrom
foxzool:master

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