Use json references instead of swagger path

Yelp / bravado-core

Other

109 stars 98 forks source link

Use json references instead of swagger path #272

Closed macisamuele closed 6 years ago

macisamuele commented 6 years ago

The goal of this PR is to replace path with json_reference during model discovery. The change allows to:

simplify the identification of models that are under [any-file]#/definitions NOTE: an additional PR will be provided to get models from all linked files (as affirmed in documentation)
improve exceptions in case of duplicated models as they will not contain the paths in swagger specs but the real references on files (extremely useful in case of multi file spec)

coveralls commented 6 years ago

Coverage increased (+0.01%) to 98.385% when pulling 3bfb5a538b53ae80506a867b466a39924d481400 on macisamuele:maci-use-json-references-instead-of-swagger-path into ada3a51369d83b5b9889c865890a80dc2433f342 on Yelp:master.