Skip to content

[KeyManager] Protoc to Buf#727

Open
atulpatildbz wants to merge 1 commit intogoogle:mainfrom
atulpatildbz:buf
Open

[KeyManager] Protoc to Buf#727
atulpatildbz wants to merge 1 commit intogoogle:mainfrom
atulpatildbz:buf

Conversation

@atulpatildbz
Copy link
Copy Markdown
Collaborator

use external protovalidate and remove vendored file

@atulpatildbz
Copy link
Copy Markdown
Collaborator Author

/gcbrun


cd "$(dirname "$0")" || exit 1

go run github.com/bufbuild/buf/cmd/buf@latest generate . --template buf.gen.yaml
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this using the buf from the latest version? Should we pin it?

Comment thread buf.lock
@@ -0,0 +1,6 @@
# Generated by buf. DO NOT EDIT.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Move this file and the one below to KeyManager directory?

// protoc-gen-go v1.36.11
// protoc v3.21.12
// source: keymanager/km_common/proto/algorithms.proto
// protoc (unknown)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These are showing up as unknown is that okay?

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