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

Ensured result is not nil. Fixes #1202. #1203

Merged
merged 1 commit into from Oct 16, 2016

Conversation

@liger1978
Copy link
Contributor

liger1978 commented Oct 9, 2016

(Also added myself to CONTRIBUTORS as I've done a few patches now)

@liger1978

This comment has been minimized.

Copy link
Contributor Author

liger1978 commented Oct 12, 2016

Any objections to this simple pull request? Don't know why the CI failed. It doesn't seem to be related to my change.

@liger1978

This comment has been minimized.

Copy link
Contributor Author

liger1978 commented Oct 14, 2016

By the way, the CI failed because the Travis Mac hosts were down: https://www.traviscistatus.com/

@jordansissel

This comment has been minimized.

Copy link
Owner

jordansissel commented Oct 16, 2016

Ahh, good fix. Thank you! <3

@jordansissel jordansissel merged commit e9a6f63 into jordansissel:master Oct 16, 2016
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.