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

fix(bazel): fix bazel build and add presubmit #1479

Merged
merged 9 commits into from Mar 22, 2024

Conversation

noahdietz
Copy link
Collaborator

@noahdietz noahdietz commented Mar 21, 2024

This does a number of things to fix the Bazel build:

  1. adds .bazeliskrc so that this repo pins to bazel 6.3.0 (like googleapis)
  2. adds bazel c++ flag to .bazelrc to fix C++ version compat issues (like googleapis)
  3. updates rules_proto to an actual released version (yes its an rc, but that's better than a commit)
  4. adds a missing any_proto dep to the proto_library target

This also adds a bazel build presubmit for the proto_library target to ensure any changes to the proto/WORKSPACE still work.

Fixes #1476

@noahdietz noahdietz marked this pull request as ready for review March 21, 2024 20:56
@noahdietz noahdietz requested review from a team as code owners March 21, 2024 20:56
@noahdietz noahdietz requested a review from parthea March 21, 2024 20:56
@noahdietz noahdietz changed the title chore(ci): add bazel build for proto_library fix(bazel): fix bazel build and add presubmit Mar 21, 2024
WORKSPACE.bazel Outdated Show resolved Hide resolved
@noahdietz noahdietz added the automerge Summon MOG for automerging label Mar 21, 2024
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Summon MOG for automerging label Mar 22, 2024
@noahdietz noahdietz merged commit e446229 into googleapis:main Mar 22, 2024
9 checks passed
@noahdietz noahdietz deleted the bazel-ci branch March 22, 2024 18:05
gcf-merge-on-green bot pushed a commit that referenced this pull request Mar 22, 2024
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.

Add a bazel integration test
2 participants