Skip to content

Conversation

@kenjones-cisco
Copy link
Contributor

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

`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

frol commented Nov 4, 2017

LGTM

@wing328 wing328 added this to the v2.3.0 milestone Nov 4, 2017
@wing328 wing328 merged commit b6699f6 into swagger-api:master Nov 4, 2017
@kenjones-cisco kenjones-cisco deleted the task/python-unicode branch November 6, 2017 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants