swagger-api / swagger-parser

Swagger Spec to Java POJOs
http://swagger.io
Apache License 2.0
781 stars 527 forks source link

Fixes issue 1975: bug in const processing, const values of all types now processable #1996

Closed spacether closed 11 months ago

spacether commented 11 months ago

Changes const type from String to Object Per json schema docs, const stores one values of any json schema type

Verification:

If merged, this will close out these issues:

spacether commented 11 months ago

@frantuma @gracekarina thank you for your work on swagger-parser. When you have some time can you review this PR? How do I get CI tests to run? The test that I added passes locally. Getting this merged in and released would help me turn on 34 json schema tests in openapi-json-schema-generator Thank you!

frantuma commented 11 months ago

Thanks!

spacether commented 11 months ago

Thanks you for the review and merge!