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 Eijebong to the list of try people #804

Conversation

@nox
Copy link
Member

nox commented Feb 19, 2018

This change is Reviewable

@SimonSapin
Copy link
Member

SimonSapin commented Feb 19, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Feb 19, 2018

📌 Commit e854b87 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Feb 19, 2018

Testing commit e854b87 with merge b27f310...

bors-servo added a commit that referenced this pull request Feb 19, 2018
…tell-bors-to-try-his-patches, r=SimonSapin

Add Eijebong to the list of try people

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

bors-servo commented Feb 19, 2018

💔 Test failed - status-travis

@notriddle
Copy link
Contributor

notriddle commented Feb 20, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Feb 20, 2018

Testing commit e854b87 with merge 0a773a9...

bors-servo added a commit that referenced this pull request Feb 20, 2018
…tell-bors-to-try-his-patches, r=SimonSapin

Add Eijebong to the list of try people

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

bors-servo commented Feb 20, 2018

💔 Test failed - status-travis

@Eijebong
Copy link
Member

Eijebong commented Feb 20, 2018

💔 @bors-servo Why do you hate me so much ?

@jdm
Copy link
Member

jdm commented Feb 20, 2018

We can retry this after #805 merges.

@jdm
Copy link
Member

jdm commented Feb 20, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Feb 20, 2018

Testing commit e854b87 with merge 1a3d1ac...

bors-servo added a commit that referenced this pull request Feb 20, 2018
…tell-bors-to-try-his-patches, r=SimonSapin

Add Eijebong to the list of try people

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

bors-servo commented Feb 20, 2018

☀️ Test successful - status-travis
Approved by: SimonSapin
Pushing 1a3d1ac to master...

@bors-servo bors-servo merged commit e854b87 into master Feb 20, 2018
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
homu Test successful
Details
@notriddle notriddle deleted the tfw-the-guy-who-does-all-the-bumps-cannot-tell-bors-to-try-his-patches branch Feb 20, 2018
@jdm jdm removed the S-needs-deploy label Feb 20, 2018
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

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