-
Notifications
You must be signed in to change notification settings - Fork 4
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 protoc-gen-openapi (from gnostic) #603
Conversation
a7ef56b
to
83899d9
Compare
83899d9
to
b4c5175
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the package name need to contain the openapi version? The tool seems to be generic so maybe sgprotocgenopenapi
is enough?
b4c5175
to
0919663
Compare
@alethenorio good one, I fixed that. |
Maybe even calling it |
The binary is |
Why this change?
So that we can generate openapiv3 JSON file from protos using
protoc-gen-openapi
and version control it from Sage.Notes
Docs: https://github.com/google/gnostic/tree/main/cmd/protoc-gen-openapi