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

Add a comment to also add trychooser branches to Treeherder #927

Merged
merged 2 commits into from Dec 13, 2018

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Dec 13, 2018

This will be relevant once mozilla/treeherder#4371 has landed.


This change is Reviewable

@jdm
Copy link
Member

jdm commented Dec 13, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2018

📌 Commit 61919dc has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2018

Testing commit 61919dc with merge dd6c47b...

bors-servo added a commit that referenced this pull request Dec 13, 2018
Add a comment to also add trychooser branches to Treeherder

This will be relevant once mozilla/treeherder#4371 has landed.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/927)
<!-- Reviewable:end -->
```
[ FAIL ] Trailing whitespace found on files and lines:
         ./homu/map.jinja:4:----
```
@SimonSapin
Copy link
Member Author

SimonSapin commented Dec 13, 2018

Removed trailing spaces.

@bors-servo r=jdm

@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2018

📌 Commit ae67cfd has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Dec 13, 2018

Testing commit ae67cfd with merge 023edcf...

bors-servo added a commit that referenced this pull request Dec 13, 2018
Add a comment to also add trychooser branches to Treeherder

This will be relevant once mozilla/treeherder#4371 has landed.

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

bors-servo commented Dec 13, 2018

☀️ Test successful - status-travis
Approved by: jdm
Pushing 023edcf to master...

@bors-servo bors-servo merged commit ae67cfd into master Dec 13, 2018
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the SimonSapin-patch-1 branch Dec 13, 2018
@jdm jdm removed the S-needs-deploy label Jan 11, 2019
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

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