Skip to content
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

Allow setting near/far clip planes #24018

Merged
merged 1 commit into from Aug 23, 2019
Merged

Conversation

@Manishearth
Copy link
Member

Manishearth commented Aug 20, 2019

needs servo/webxr#41

r? @asajeffrey


This change is Reviewable

@highfive
Copy link

highfive commented Aug 20, 2019

Heads up! This PR modifies the following files:

@highfive
Copy link

highfive commented Aug 20, 2019

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!
@asajeffrey
Copy link
Member

asajeffrey commented Aug 21, 2019

You can r=me once webxr is updated, and the Cargo.lock file is regenerated.

@Manishearth Manishearth force-pushed the Manishearth:wherever-you-are branch from ebbb5d9 to a5575da Aug 22, 2019
@Manishearth
Copy link
Member Author

Manishearth commented Aug 22, 2019

@bors-servo r=asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Aug 22, 2019

📌 Commit a5575da has been approved by asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2019

Testing commit a5575da with merge dfad788...

bors-servo added a commit that referenced this pull request Aug 23, 2019
Allow setting near/far clip planes

needs servo/webxr#41

r? @asajeffrey

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24018)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2019

💔 Test failed - status-taskcluster

@jdm
Copy link
Member

jdm commented Aug 23, 2019

Needs a mach fmt.

@Manishearth Manishearth force-pushed the Manishearth:wherever-you-are branch from a5575da to 0d79b2b Aug 23, 2019
@Manishearth
Copy link
Member Author

Manishearth commented Aug 23, 2019

@bors-servo r=asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2019

📌 Commit 0d79b2b has been approved by asajeffrey

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2019

Testing commit 0d79b2b with merge f21d116...

bors-servo added a commit that referenced this pull request Aug 23, 2019
Allow setting near/far clip planes

needs servo/webxr#41

r? @asajeffrey

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24018)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: asajeffrey
Pushing f21d116 to master...

@bors-servo bors-servo merged commit 0d79b2b into servo:master Aug 23, 2019
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.