[python] Convert unicode to six.u #6881

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

Jump to bottom

Merged

wing328 merged 1 commit into swagger-api:master from kenjones-cisco:task/python-unicode

Nov 4, 2017

Merged

[python] Convert unicode to six.u #6881

wing328 merged 1 commit into swagger-api:master from kenjones-cisco:task/python-unicode

Nov 4, 2017

Conversation

@kenjones-cisco

Copy link

Contributor

@kenjones-cisco kenjones-cisco commented Nov 4, 2017

PR checklist

Read the contribution guidelines.
Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\.
Filed the PR against the correct branch: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
Copied the technical committee to review the pull request if your PR is targeting a particular programming langauge.

Description of the PR

(details of the change, additional tests that have been done, reference to the issue for tracking, etc)

unicode does not exist in py3, therefore since six is already imported
switch to using six.u so that unicode is used on py2 and str is used
on py3.

@taxpon @frol @mbohlool @cbornet
CC @wing328

@kenjones-cisco


 [python] Convert unicode to six.u

6cea9b9

`unicode` does not exist in py3, therefore since six is already imported
switch to using `six.u` so that `unicode` is used on py2 and `str` is used
on py3.

@frol

Copy link

Contributor