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

homu: Give @CYBAI try access. #775

Merged
merged 1 commit into from Jan 13, 2018
Merged

homu: Give @CYBAI try access. #775

merged 1 commit into from Jan 13, 2018

Conversation

@emilio
Copy link
Member

emilio commented Jan 13, 2018

He's done a ton of work and I thought he already had it :).


This change is Reviewable

He's done a ton of work and I thought he already had it :).
@emilio
Copy link
Member Author

emilio commented Jan 13, 2018

r? @jdm or anyone else

@highfive highfive assigned jdm and unassigned aneeshusa Jan 13, 2018
@KiChjang
Copy link
Member

KiChjang commented Jan 13, 2018

I agree!

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2018

📌 Commit d4dd844 has been approved by KiChjang

@highfive highfive assigned KiChjang and unassigned jdm Jan 13, 2018
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2018

Testing commit d4dd844 with merge 81e0bdf...

bors-servo added a commit that referenced this pull request Jan 13, 2018
homu: Give @CYBAI try access.

He's done a ton of work and I thought he already had it :).

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

bors-servo commented Jan 13, 2018

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

@bors-servo bors-servo merged commit d4dd844 into master Jan 13, 2018
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@jdm jdm removed the S-needs-deploy label Jan 15, 2018
@jdm
Copy link
Member

jdm commented Jan 15, 2018

This change has been deployed.

@CYBAI CYBAI mentioned this pull request Jul 5, 2019
4 of 4 tasks complete
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.