reverb / swagger-spec

This fork of the swagger-spec is intended for 2.0 planning
Other
19 stars 15 forks source link

Potential naming mismatch: 'schemas' refer to 'definitions' #108

Closed earth2marsh closed 10 years ago

earth2marsh commented 10 years ago

At one point we considered calling what is now called definitions schemas instead. We opted for definitions, but the operation declaration still uses a key called schema. Perhaps that should be changed to definition for a better match?

cursor_and_a_swagger_yaml_specification_for_uber_s_new_api

fehguy commented 10 years ago

I think this is OK. The #/definitions is JSON schema convention, and inside that location, a schema exists. So referring from schema to the definitions object is fine.