-
Notifications
You must be signed in to change notification settings - Fork 9
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
Support doseq for urlencode #14
Conversation
@@ -30,10 +30,16 @@ def _pairs(obj): | |||
return obj.iteritems() if isinstance(obj, dict) else obj | |||
|
|||
|
|||
def urlencode(query, *args, **kwargs): | |||
def urlencode(query, doseq=0): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why a 0 instead of False
? May be a docstring for doseq
as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just copying what the stdlib does, not sure why it's done that way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically you can reassign True in 2.x, but not 0. So there IS that, hah
lgtm. |
import urllib | ||
import urlparse | ||
|
||
from yelp_uri import urllib_utf8 | ||
|
||
|
||
_suites = ['unicode'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
heh, leftovers from testify days, can delete
testing: 🐵 |
Resolves #13
I ran into some flake8 errors, which are the other changes here.