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

Fix issue 389 #390

Merged
merged 3 commits into from Oct 21, 2013

Conversation

Projects
None yet
2 participants
@hydrogen18
Copy link
Contributor

hydrogen18 commented Oct 19, 2013

No description provided.

hydrogen18 added some commits Oct 19, 2013

Update client.py
Check for `None` instead of relying on the truth value of whatever the user happens to pass in for `unix_socket_path`. I think it is preferable to blow up when passed something like `0` rather than silently making a TCP connection.
Update client.py
put spaces around operator
Update client.py
PEP 8

andymccurdy added a commit that referenced this pull request Oct 21, 2013

@andymccurdy andymccurdy merged commit f95410b into andymccurdy:master Oct 21, 2013

1 check passed

default The Travis CI build passed
Details
@andymccurdy

This comment has been minimized.

Copy link
Owner

andymccurdy commented Oct 21, 2013

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment