Skip to content

refactor(ci): install Bazel using setup-bazel#3494

Merged
chaokunyang merged 1 commit intoapache:mainfrom
BaldDemian:bazel-ci
Mar 19, 2026
Merged

refactor(ci): install Bazel using setup-bazel#3494
chaokunyang merged 1 commit intoapache:mainfrom
BaldDemian:bazel-ci

Conversation

@BaldDemian
Copy link
Contributor

@BaldDemian BaldDemian commented Mar 19, 2026

Why?

  • Setting up Bazel manually cannot handle network issues properly.
  • old rules_cc version will cause one job failure on the latest macOS runner.

What does this PR do?

  • Setting up Bazel using GitHub action across CI.
  • Bump rules_cc version.

Related issues

No.

AI Contribution Checklist

No.

Does this PR introduce any user-facing change?

No.

Benchmark

No.

@BaldDemian
Copy link
Contributor Author

Hi @chaokunyang, maybe you should enable setup-bazel manually in the settings of Fory.
https://github.com/apache/fory/actions/runs/23279089056

Copy link
Collaborator

@chaokunyang chaokunyang left a comment

Choose a reason for hiding this comment

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

Looks great

@chaokunyang chaokunyang merged commit 847087a into apache:main Mar 19, 2026
3 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.

2 participants