Skip to content

Conitnue fixing of #2037#2063

Merged
wing328 merged 2 commits intoswagger-api:masterfrom
mtngld:issue_2037
Feb 8, 2016
Merged

Conitnue fixing of #2037#2063
wing328 merged 2 commits intoswagger-api:masterfrom
mtngld:issue_2037

Conversation

@mtngld
Copy link
Contributor

@mtngld mtngld commented Feb 8, 2016

This PR is additional work on #2037, and is a fixed version of addition made in #2058.

  1. After creating Add model's "additionalProperties" handling in python client #2058 to comply with python 3.4 found a bug when using on python 2.7, the current method should comply with both.
  2. Performance optimization by switching the order of the elif, since I guess additionalProperties is probably less used and should be checked last
  3. Style optimization by warping lines and staying below 79 chars

@wing328 wing328 added this to the v2.1.6 milestone Feb 8, 2016
wing328 added a commit that referenced this pull request Feb 8, 2016
@wing328 wing328 merged commit 2a73c11 into swagger-api:master Feb 8, 2016
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.

2 participants