WIP: Add basic support for openapi

[gabrieltaylor]

Adds support for PetStore OpenAPI 3 example.

Support for OpenAPI 3 is not feature complete.

TODO

• Add support for callbacks ✅
• Add support for links example ✅
• Add support for examples example ✅
• Add support for authentication schemes ✅
• Expand tests

[gabrieltaylor]

@fotinakis I think that this is ready for your review. Looking forward to hearing your thoughts.

[haseleyi]

Would love to see this feature supported :)

[auscaster]

It seems like a lot of work has gone into this, any chance of getting it merged?

[pascalbetz]

Or any feedback on why it has not been merged so far?
Is there something we can do?
What is missing? What could/should be done differently?

We use swagger-blocks and this PR contains features that would be useful to us (the gem is already useful to us. Even without this PR. Many thanks).

[fotinakis]

LGTM 👍

Thanks for putting this together. I don't have much time to maintain this gem anymore, but can merge and release this. Should this be a major version bump or is it backwards compatible?

[gabrieltaylor]

@fotinakis It should be backwards compatible as the v2 tests still pass, however, given the number of changes and the fact that it adds support for a major version change of Open API, a major version bump of swagger-blocks might be safer.

[fotinakis]

Thanks for the great work here.

Released as a major bump to gem version 3.0.0 — if there bugs we can fix them and release there.

[auscaster]

Nice one Mike! Thanks for merging :)

…

On Thu, 26 Sep 2019 at 19:56, Mike Fotinakis ***@***.***> wrote: Thanks for the great work here. Released as a major bump to gem version 3.0.0 — if there bugs we can fix them and release there. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#120?email_source=notifications&email_token=AAAYUDBPTBSRSB575MF6UJDQLTZTBA5CNFSM4FZUVF7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7WN7IQ#issuecomment-535617442>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAYUDFSNJ7HDSHPBFCXIWTQLTZTBANCNFSM4FZUVF7A> .

[PhilippeChab]

The README should be updated to reflect this merge.