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

Don't crash if username field is missing. #92

Merged
merged 1 commit into from Aug 6, 2015

Conversation

@metajack
Copy link
Contributor

metajack commented Aug 2, 2015

Review on Reviewable

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Aug 5, 2015

@metajack
Copy link
Contributor Author

metajack commented Aug 6, 2015

@bors-servo r=larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2015

📌 Commit 72b246f has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2015

Testing commit 72b246f with merge 7624a2d...

bors-servo pushed a commit that referenced this pull request Aug 6, 2015
…strom

Don't crash if username field is missing.



<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/92)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit 72b246f into servo:master Aug 6, 2015
1 of 2 checks passed
1 of 2 checks passed
homu Testing commit 72b246f with merge 7624a2d...
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@metajack metajack deleted the metajack:github-listener-username-fix branch Aug 6, 2015
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

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