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 annotations.proto and generated Go code #5

Merged
merged 1 commit into from
Aug 4, 2023
Merged

Add annotations.proto and generated Go code #5

merged 1 commit into from
Aug 4, 2023

Conversation

pkwarren
Copy link
Contributor

In order to prepare a PR for
google/gnostic#397, gnostic-models needs to be updated to include openapiv3/annotations.proto (and its generated code).

In order to prepare a PR for
google/gnostic#397, gnostic-models needs to be
updated to include openapiv3/annotations.proto (and its generated code).
// Code generated by protoc-gen-go. DO NOT EDIT.
// versions:
// protoc-gen-go v1.27.1
// protoc v3.19.3
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generated this with the same tool versions used to generate openapiv3/OpenAPIv3.proto code.

import "openapiv3/OpenAPIv3.proto";

// The Go package name.
option go_package = "./openapiv3;openapi_v3";
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only change to this file (matching the one made to OpenAPIv3.proto).

@pkwarren
Copy link
Contributor Author

pkwarren commented Aug 2, 2023

@timburks - Are you the right person to tag for reviews on these PRs? I was hoping to get some feedback on this and the downstream fix in google/gnostic#400 but happy to ping another reviewer.

@timburks
Copy link
Contributor

timburks commented Aug 4, 2023

@pkwarren Thanks, AFAIK this is only used by @Jefftree and the k8s team, so since they're ok with google/gnostic#397, I think it's fine to proceed with this. Merging...

@timburks timburks merged commit c7be7c7 into google:main Aug 4, 2023
@pkwarren pkwarren deleted the pkw/add-annotations-proto branch August 6, 2023 04:57
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