search

cyprieng / swagger-parser

Give useful informations about your swagger files
MIT License
62 stars 59 forks source link

Add all of support #61

Closed rwincewicz closed 6 years ago

rwincewicz commented 6 years ago

I've added support for the allOf property. The parser checks for an allOf section and, if it finds one, iterates through each of the items in the list feeding them back through the get_example_from_prop_spec method. In this case a from_allof flag is set so that the resultant example is returned as a dictionary that can be added to the other components of the example. I've also updated the tests to check that the different examples of allOf sections are parsed correctly.

flavianh commented 6 years ago

@cyprien-g Not sure that we need python3.3 support in the CI, what do you think?

flavianh commented 6 years ago

@rwincewicz Could you remove py33 in https://github.com/Trax-air/swagger-parser/blob/master/tox.ini ? I don't think we absolutely need to run python 3.3 in the CI. Should make your tests pass

cyprieng commented 6 years ago

@rwincewicz can you make the same changes than this commit (https://github.com/Trax-air/swagger-stub/commit/940de2c188d3306f50a638deabcbc8c6bbb9b015) on the setup.py ?

rwincewicz commented 6 years ago

@cyprieng Cool, thanks for the pointer. That's the change made and everything is passing.