Skip to content

ci: use BuildBuddy for rust-ci-full non-Windows argument-comment-lint#16136

Merged
bolinfest merged 1 commit intomainfrom
pr16136
Mar 28, 2026
Merged

ci: use BuildBuddy for rust-ci-full non-Windows argument-comment-lint#16136
bolinfest merged 1 commit intomainfrom
pr16136

Conversation

@bolinfest
Copy link
Copy Markdown
Collaborator

@bolinfest bolinfest commented Mar 28, 2026

Why

PR #16130 fixed the Windows argument-comment-lint regression in rust-ci-full, but the next main runs still left the Linux and macOS lint legs timing out.

In run 23695263729, both non-Windows argument-comment-lint jobs were cancelled almost exactly 30 minutes after they started. The remaining workflow difference versus rust-ci.yml was that rust-ci-full did not pass BUILDBUDDY_API_KEY into the non-Windows Bazel lint step, so run-bazel-ci.sh fell back to local Bazel configuration instead of using the faster remote-backed path available on main.

What changed

Test plan

  • loaded .github/workflows/rust-ci-full.yml and .github/workflows/rust-ci.yml with python3 + yaml.safe_load(...)
  • inspected run 23695263729 and confirmed Argument comment lint - Linux and Argument comment lint - macOS were cancelled about 30 minutes after start
  • verified the updated rust-ci-full step now matches the non-Windows secret wiring already present in rust-ci.yml

References

@bolinfest bolinfest changed the title ci: use BuildBuddy for rust-ci-full argument-comment-lint Mar 28, 2026
@bolinfest bolinfest merged commit a16a910 into main Mar 28, 2026
21 checks passed
@bolinfest bolinfest deleted the pr16136 branch March 28, 2026 22:36
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

1 participant