Skip to content

Comments

chore: re-generate code and add CI check#363

Merged
sluongng merged 1 commit intobazelbuild:mainfrom
tyler-french:tfrench/gencode
Feb 10, 2026
Merged

chore: re-generate code and add CI check#363
sluongng merged 1 commit intobazelbuild:mainfrom
tyler-french:tfrench/gencode

Conversation

@tyler-french
Copy link
Contributor

@tyler-french tyler-french commented Feb 10, 2026

Verify CI correctly fails in #364

@tyler-french tyler-french force-pushed the tfrench/gencode branch 2 times, most recently from 1def347 to 3e7bb1c Compare February 10, 2026 15:46
@tyler-french tyler-french marked this pull request as ready for review February 10, 2026 16:01
Copilot AI review requested due to automatic review settings February 10, 2026 16:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR re-generates Go protobuf bindings for the Remote Execution API and updates the GitHub Actions CI workflow to verify generated code is committed and up to date.

Changes:

  • Regenerated Go protobuf/grpc bindings to reflect updated REAPI definitions (digest + chunking support, updated CAS SplitBlob/SpliceBlob docs).
  • Updated CI to run the repo’s generation hook and fail when it produces diffs.
  • Updated CI workflow triggers/checkout action version.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
build/bazel/remote/execution/v2/remote_execution_grpc.pb.go Regenerated gRPC client/server interfaces and updated RPC documentation for CAS chunking APIs.
build/bazel/remote/execution/v2/remote_execution.pb.go Regenerated protobuf types/enums; adds Git SHA1 digest option and chunking-related enums/messages/fields.
.github/workflows/main.yml Updates CI triggers/checkout action and adds a generated-code freshness check.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sluongng sluongng merged commit 3ab1150 into bazelbuild:main Feb 10, 2026
2 checks passed
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.

3 participants