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

Build docs and upload to doc.servo.org on Taskcluster, add support for trychoosers #22381

Merged
merged 8 commits into from Dec 12, 2018

Conversation

Projects
None yet
6 participants
@SimonSapin
Copy link
Member

SimonSapin commented Dec 7, 2018

This change is Reviewable

@highfive

This comment has been minimized.

Copy link

highfive commented Dec 7, 2018

Heads up! This PR modifies the following files:

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 7, 2018

@highfive highfive assigned Manishearth and unassigned nox Dec 7, 2018

@SimonSapin SimonSapin referenced this pull request Dec 7, 2018

Open

Meta: move CI from Buildbot to Taskcluster #22325

15 of 24 tasks complete
@jdm

This comment has been minimized.

Copy link
Member

jdm commented Dec 7, 2018

@SimonSapin So http://doc.servo.org/ no longer contains any content. Is that expected?

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 8, 2018

That is not exactly expected of course, but it is explained by the "live" test that I did of pushing with the secret token, before adding the CNAME file and others from etc/doc.servo.org. I expect it’ll be fixed when this PR lands (or next time the daily job runs on Buildbot), but I’ll fix it manually in the meantime.

@SimonSapin SimonSapin force-pushed the tc-rustdoc branch from 949841e to 7918143 Dec 8, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 8, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 12, 2018

☔️ The latest upstream changes (presumably #22419) made this pull request unmergeable. Please resolve the merge conflicts.

@SimonSapin SimonSapin force-pushed the tc-rustdoc branch from 4faf9e7 to 8588e92 Dec 12, 2018

@SimonSapin SimonSapin changed the title Build docs and upload to doc.servo.org on Taskcluster Build docs and upload to doc.servo.org on Taskcluster, add support for trychoosers Dec 12, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 12, 2018

I sneaked support for trychoosers in this PR.

@SimonSapin SimonSapin force-pushed the tc-rustdoc branch from 6143d64 to bdac1f6 Dec 12, 2018

@SimonSapin SimonSapin force-pushed the tc-rustdoc branch from bdac1f6 to e7b493d Dec 12, 2018

@SimonSapin

This comment has been minimized.

Copy link
Member

SimonSapin commented Dec 12, 2018

@bors-servo r=nox

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 12, 2018

📌 Commit e7b493d has been approved by nox

@highfive highfive assigned nox and unassigned Manishearth Dec 12, 2018

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented Dec 12, 2018

⌛️ Testing commit e7b493d with merge a86c23c...

bors-servo added a commit that referenced this pull request Dec 12, 2018

Auto merge of #22381 - servo:tc-rustdoc, r=nox
Build docs and upload to doc.servo.org on Taskcluster, add support for trychoosers

<!-- 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/22381)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

@bors-servo bors-servo merged commit e7b493d into master Dec 12, 2018

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details

@SimonSapin SimonSapin deleted the tc-rustdoc branch Dec 12, 2018

SimonSapin added a commit that referenced this pull request Dec 12, 2018

bors-servo added a commit that referenced this pull request Dec 13, 2018

Auto merge of #22436 - servo:tc-master, r=SimonSapin
Also run Taskcluster tasks when pushing to `master`

I forgot to include this in #22381

<!-- 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/22436)
<!-- Reviewable:end -->

bors-servo added a commit that referenced this pull request Dec 13, 2018

Auto merge of #22436 - servo:tc-master, r=SimonSapin
Also run Taskcluster tasks when pushing to `master`

I forgot to include this in #22381

<!-- 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/22436)
<!-- Reviewable:end -->

bors-servo added a commit that referenced this pull request Dec 13, 2018

Auto merge of #22436 - servo:tc-master, r=SimonSapin
Also run Taskcluster tasks when pushing to `master`

I forgot to include this in #22381

<!-- 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/22436)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment