Skip to content

Commit

Permalink
Update bazel-diff from 4.5 to 8.1.1... (bazelbuild#2088)
Browse files Browse the repository at this point in the history
...and invoke it with --excludeExternalTargets so that it works properly
in the presence of --noenable_workspace.
  • Loading branch information
fweikert authored Oct 28, 2024
1 parent 0447f7a commit 6829be2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion buildkite/bazelci.py
Original file line number Diff line number Diff line change
Expand Up @@ -588,7 +588,7 @@

# TODO(fweikert): Install bazel-diff on the Docker images and on the Mac machines
BAZEL_DIFF_URL = (
"https://github.com/Tinder/bazel-diff/releases/download/4.5.0/bazel-diff_deploy.jar"
"https://github.com/Tinder/bazel-diff/releases/download/8.1.1/bazel-diff_deploy.jar"
)

AUTO_DIFFBASE_VALUES = frozenset(["1", "true", "auto"])
Expand Down Expand Up @@ -2579,6 +2579,7 @@ def run_bazel_diff(bazel_diff_path, old_workspace_dir, new_workspace_dir, bazel_
"-jar",
bazel_diff_path,
"generate-hashes",
"--excludeExternalTargets",
"-w",
repo_dir,
"-b",
Expand Down

0 comments on commit 6829be2

Please sign in to comment.