Skip to content

Commit

Permalink
Fix Intellij aspect integration tests setup
Browse files Browse the repository at this point in the history
Adds a dependency on the minimal `@rules_java` version that has the required top level symbols.

Also drops testing against Bazel 4 (it is no longer supported, as of Jan 2024, see: https://bazel.build/release#support-matrix)

Should turn CI green and unblock the release.

Tested: https://buildkite.com/bazel/intellij-plugin-aspect-google/builds/9062
PiperOrigin-RevId: 627296165
  • Loading branch information
Googler authored and copybara-github committed Apr 23, 2024
1 parent da65176 commit ee1be83
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ java_binary(
bazel_integration_tests(
name = "bazel_invocation_integration_tests",
bazel_versions = [
"4.0.0",
"6.0.0",
],
# set tags = [] because otherwise bazel_integration_tests sets
Expand Down
Original file line number Diff line number Diff line change
@@ -1 +1,8 @@
workspace(name = "testdata")

load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
http_archive(
name = "rules_java",
url = "https://github.com/bazelbuild/rules_java/releases/download/5.3.5/rules_java-5.3.5.tar.gz",
sha256 = "c73336802d0b4882e40770666ad055212df4ea62cfa6edf9cb0f9d29828a0934",
)

0 comments on commit ee1be83

Please sign in to comment.