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

Bump euclid and heapsize and bump version #1303

Merged
merged 1 commit into from May 30, 2017
Merged

Conversation

@Eijebong
Copy link
Member

Eijebong commented May 29, 2017


This change is Reviewable

@Eijebong Eijebong force-pushed the Eijebong:master branch from 65eaa95 to 3f83bcb May 29, 2017
Copy link
Member

kvark left a comment

Looks good, just one question

@@ -17,7 +17,7 @@ app_units = "0.4"
bincode = "1.0.0-alpha6"
bit-set = "0.4"
byteorder = "1.0"
euclid = "0.11.2"
euclid = ">= 0.11, < 0.14"

This comment has been minimized.

@kvark

kvark May 29, 2017

Member

I'm not 100% sure about this being a non-breaking change. Is this approach established elsewhere for a reference?

This comment has been minimized.

@Eijebong

Eijebong May 29, 2017

Author Member

Well I didn't change any code so unless you're reexporting euclid somewhere I don't think that this is a breaking change.

This comment has been minimized.

@Eijebong

Eijebong May 29, 2017

Author Member

If you want I can force euclid 0.13 and bump to 0.40.0

@Eijebong Eijebong force-pushed the Eijebong:master branch from 46ae8ae to b9da97e May 29, 2017
@kvark
Copy link
Member

kvark commented May 30, 2017

@glennw heads up - this may prevent updating WR for servo until everything is using euclid 0.13
@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented May 30, 2017

📌 Commit b9da97e has been approved by kvark

@bors-servo
Copy link
Contributor

bors-servo commented May 30, 2017

Testing commit b9da97e with merge 5b56f1c...

bors-servo added a commit that referenced this pull request May 30, 2017
Bump euclid and heapsize and bump version

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

bors-servo commented May 30, 2017

☀️ Test successful - status-travis
Approved by: kvark
Pushing 5b56f1c to master...

@bors-servo bors-servo merged commit b9da97e into servo:master May 30, 2017
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
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

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