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

Give jonleighton try access. #826

Merged
merged 1 commit into from May 3, 2018
Merged

Give jonleighton try access. #826

merged 1 commit into from May 3, 2018

Conversation

@emilio
Copy link
Member

emilio commented Apr 26, 2018

He's done a bunch of contributions and I'm tired of sending the patches to bors, so this should ensure he can do it :P


This change is Reviewable

He's done a bunch of contributions and I'm tired of sending the patches to bors, so this should ensure he can do it :P
@KiChjang
Copy link
Member

KiChjang commented Apr 26, 2018

Let's get this rolling!

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Apr 26, 2018

📌 Commit c22e74f has been approved by KiChjang

@bors-servo
Copy link
Contributor

bors-servo commented Apr 26, 2018

Testing commit c22e74f with merge 9349478...

bors-servo added a commit that referenced this pull request Apr 26, 2018
Give jonleighton try access.

He's done a bunch of contributions and I'm tired of sending the patches to bors, so this should ensure he can do it :P

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

bors-servo commented Apr 26, 2018

💔 Test failed - status-travis

@jonleighton
Copy link
Contributor

jonleighton commented Apr 28, 2018

Anyone know what to do about that?

@jdm jdm mentioned this pull request May 3, 2018
@jdm
Copy link
Member

jdm commented May 3, 2018

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented May 3, 2018

Testing commit c22e74f with merge f3bd5b9...

bors-servo added a commit that referenced this pull request May 3, 2018
Give jonleighton try access.

He's done a bunch of contributions and I'm tired of sending the patches to bors, so this should ensure he can do it :P

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

bors-servo commented May 3, 2018

☀️ Test successful - status-travis
Approved by: KiChjang
Pushing f3bd5b9 to master...

@bors-servo bors-servo merged commit c22e74f into master May 3, 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
@jdm jdm removed the S-needs-deploy label May 4, 2018
@jdm
Copy link
Member

jdm commented May 4, 2018

Deployed to master.

@jonleighton
Copy link
Contributor

jonleighton commented May 5, 2018

Thanks!

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

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