Skip to content

/ saltfs

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 gw3583 to reviewers list. #832

Merged
merged 1 commit into from May 7, 2018
Merged

Add gw3583 to reviewers list. #832

merged 1 commit into from May 7, 2018

Conversation

@gw3583
Copy link
Contributor

gw3583 commented May 7, 2018 •
edited by larsbergstrom

This change is Reviewable
@gw3583
Add gw3583 to reviewers list.
Loading status checks…
f8e6165
@gw3583
Copy link
Contributor Author

gw3583 commented May 7, 2018

Through an unfortunate sequence of events, I'm currently unable to log in to my normal @glennw account (for at least the next two weeks). So, I'm adding a new account to use in the interim.

r? @edunham
@edunham
Copy link
Contributor

edunham commented May 7, 2018

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2018

📌 Commit f8e6165 has been approved by edunham
@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2018

⌛ Testing commit f8e6165 with merge 4b845db...
bors-servo added a commit that referenced this pull request May 7, 2018
@bors-servo
Auto merge of #832 - gw3583:new-user, r=edunham
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
4b845db 
Add gw3583 to reviewers list.

<!-- 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/832)
<!-- Reviewable:end -->
@edunham
Copy link
Contributor

edunham commented May 7, 2018

I'm verbally confirming with the real-world instance of Glenn that this change is correct before Salting it out :)
@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2018

☀️ Test successful - status-travis
Approved by: edunham
Pushing 4b845db to master...
@bors-servo bors-servo merged commit f8e6165 into servo:master May 7, 2018
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@gw3583 gw3583 deleted the gw3583:new-user branch May 7, 2018
@jdm jdm removed the S-needs-deploy label May 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@edunham edunham

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
@gw3583 @edunham @bors-servo @jdm @highfive
You can’t perform that action at this time.