Skip to content

/ servo

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

style: Update rayon. #18728

Merged
merged 1 commit into from Oct 4, 2017
Merged

style: Update rayon. #18728

merged 1 commit into from Oct 4, 2017

Conversation

@emilio
Copy link
Member

emilio commented Oct 3, 2017 •
edited by SimonSapin

It has a couple soundness fixes.

This change is Reviewable
@emilio
style: Update rayon.
Unverified
This user has not uploaded their public key yet.
GPG key ID: 056B727BB9C1027C Learn about signing commits
Loading status checks…
a08ab7b 
It has a couple soundness fixes.
@highfive
Copy link

highfive commented Oct 3, 2017

Heads up! This PR modifies the following files:
@highfive
Copy link

highfive commented Oct 3, 2017

warning Warning warning

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

emilio commented Oct 3, 2017

r? @jdm (or anyone else)
@highfive highfive assigned jdm and unassigned mbrubeck Oct 3, 2017
@jdm
Copy link
Member

jdm commented Oct 3, 2017

@bors-servo: r+
@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2017

📌 Commit a08ab7b has been approved by jdm
@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2017

⌛ Testing commit a08ab7b with merge f8e82d8ecdbb29b797942c650a96c79980a3902f...
@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2017

⛄ The build was interrupted to prioritize another pull request.
@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2017

⌛ Testing commit a08ab7b with merge ced5fb7...
bors-servo added a commit that referenced this pull request Oct 3, 2017
@bors-servo
Auto merge of #18728 - emilio:rayonup, r=jdm
Loading status checks…
ced5fb7 
style: Update rayon.

It has a couple soundness fixes.

<!-- 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/18728)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2017

💔 Test failed - windows-msvc-dev
@mbrubeck
Copy link
Contributor

mbrubeck commented Oct 3, 2017

@bors-servo retry

  • Spurious failure from interrupt.
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

⌛ Testing commit a08ab7b with merge 3b0930c18b2f30e2ba035d4c14f540a91380714e...
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

💔 Test failed - windows-msvc-dev
@mbrubeck
Copy link
Contributor

mbrubeck commented Oct 4, 2017

@bors-servo retry

  • One of the Windows build machines is in a bad state
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

⌛ Testing commit a08ab7b with merge 6e4f1ac...
bors-servo added a commit that referenced this pull request Oct 4, 2017
@bors-servo
Auto merge of #18728 - emilio:rayonup, r=jdm
Loading status checks…
6e4f1ac 
style: Update rayon.

It has a couple soundness fixes.

<!-- 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/18728)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

💔 Test failed - windows-msvc-dev
@KiChjang
Copy link
Member

KiChjang commented Oct 4, 2017

@bors-servo retry infra
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

⌛ Testing commit a08ab7b with merge a12435f...
bors-servo added a commit that referenced this pull request Oct 4, 2017
@bors-servo
Auto merge of #18728 - emilio:rayonup, r=jdm
Loading status checks…
a12435f 
style: Update rayon.

It has a couple soundness fixes.

<!-- 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/18728)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

💔 Test failed - windows-msvc-dev
@mbrubeck
Copy link
Contributor

mbrubeck commented Oct 4, 2017

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

⌛ Testing commit a08ab7b with merge a4f6421...
bors-servo added a commit that referenced this pull request Oct 4, 2017
@bors-servo
Auto merge of #18728 - emilio:rayonup, r=jdm
Loading status checks…
a4f6421 
style: Update rayon.

It has a couple soundness fixes.

<!-- 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/18728)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 4, 2017

☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev
Approved by: jdm
Pushing a4f6421 to master...
@bors-servo bors-servo merged commit a08ab7b into servo:master Oct 4, 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
@bors-servo bors-servo mentioned this pull request Oct 4, 2017
Merged
7 of 15 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jdm jdm

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
@emilio @highfive @jdm @bors-servo @mbrubeck @KiChjang
You can’t perform that action at this time.