Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add solo-io/protoc-gen-openapi plugin to support OpenAPI V3 #418

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

leonteq-reisg
Copy link

No description provided.

@strophy
Copy link
Collaborator

strophy commented Jan 9, 2025

Hi @leonteq-reisg thanks for contributing this! Out of curiosity, how does it differ from OpenAPI v2?

It seems the build currently fails in CI due to some issue with the interaction between make, go and xx. Please see if you can fix it, otherwise I'll be back from holiday next week and can have a try as well.

@leonteq-reisg
Copy link
Author

Hi @strophy, the differences are nicely described here: https://blog.stoplight.io/difference-between-open-v2-v3-v31, but we're mainly interested in better handling of the keyword oneof when generating the examples.

I'll check the build failure and try to fix it...

@leonteq-reisg
Copy link
Author

Not entirely sure this is correct, but I remove the xx-go --wrap call call...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants