Skip to content

chore: Don't use EOL OSes in BCR presubmit#87

Merged
alexeagle merged 1 commit intobazel-contrib:mainfrom
hofbi:patch-2
Nov 26, 2025
Merged

chore: Don't use EOL OSes in BCR presubmit#87
alexeagle merged 1 commit intobazel-contrib:mainfrom
hofbi:patch-2

Conversation

@hofbi
Copy link
Copy Markdown
Contributor

@hofbi hofbi commented Nov 26, 2025

No description provided.

@hofbi hofbi changed the title Don't use EOL OSes in BCR presubmit chore: Don't use EOL OSes in BCR presubmit Nov 26, 2025
@aspect-workflows
Copy link
Copy Markdown

aspect-workflows Bot commented Nov 26, 2025

Test (Bazel 7.x) (Test)

All tests were cache hits

24 tests (100.0%) were fully cached saving 2s.


Test (Bazel 8.x) (Test)

All tests were cache hits

24 tests (100.0%) were fully cached saving 2s.


Test (Bazel 9.x) (Test)

All tests were cache hits

24 tests (100.0%) were fully cached saving 2s.


Test (Bazel 7.x) (Test)

e2e/smoke

All tests were cache hits

1 test (100.0%) was fully cached saving 32ms.


Test (Bazel 8.x) (Test)

e2e/smoke

All tests were cache hits

1 test (100.0%) was fully cached saving 63ms.


Test (Bazel 9.x) (Test)

e2e/smoke

All tests were cache hits

1 test (100.0%) was fully cached saving 45ms.


Test (WORKSPACE) (Test)

e2e/smoke

All tests were cache hits

1 test (100.0%) was fully cached saving 59ms.


Test (Bazel 7.x) (Test)

examples

All tests were cache hits

3 tests (100.0%) were fully cached saving 135ms.


Test (Bazel 8.x) (Test)

examples

All tests were cache hits

3 tests (100.0%) were fully cached saving 158ms.

@chatgpt-codex-connector
Copy link
Copy Markdown

💡 Codex Review

tar.bzl/tar/private/tar.bzl

Lines 161 to 164 in ac1d2b9

"awk_script": attr.label(
allow_single_file = True,
default = "@tar.bzl//lib/private:modify_mtree.awk",
doc = "Path to an AWK script used to modify the mtree file. By default, it uses the modify_mtree.awk script.",

P1 Badge Correct default awk_script label for mtree_mutate

The default awk_script now points to @tar.bzl//lib/private:modify_mtree.awk, but this repository only contains the script at //tar/private:modify_mtree.awk (exported in tar/private/BUILD). With the current label every mtree_mutate run that relies on the default fails to load the script unless callers override the attribute, breaking the default mutate path.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@alexeagle alexeagle merged commit d2a02b0 into bazel-contrib:main Nov 26, 2025
6 of 7 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