search

krakenjs / swaggerize-routes

Swagger document driven route builder.
Other
58 stars 57 forks source link

Move to Swagger 2.0 #6

Closed tlivings closed 10 years ago

tlivings commented 10 years ago

See: Swagger 2.0 Spec

aredridel commented 10 years ago

This looks reasonable aside from the censoring the npm package so heavily.

aredridel commented 10 years ago

How are errors in the swagger doc handled? Any intelligence there?

tlivings commented 10 years ago

The first thing that happens is the document is validated.

pvenkatakrishnan commented 10 years ago

i'm not completely familiar with changes between swagger 2.0 and swagger 1.2 so i cant review towards that migration. But the code changes by themselves look good to me.