Skip to content

Commit c976df9

Browse files
committed
Run test_lint.sh in CI, src/ in test_rules_scala
`test_lint.sh` now contains tests to lint `MODULE.bazel` files, so this runs the script in CI instead of just //tools:lint_check. Adds `src/...` to the `bazel build` and `bazel test` commands in `test_rules_scala.sh` to follow up on bazel-contrib#1719 and bazel-contrib#1721.
1 parent 554a2d1 commit c976df9

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

.bazelci/presubmit.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -102,10 +102,10 @@ tasks:
102102
shell_commands:
103103
- "./test_cross_build.sh"
104104
lint_linux:
105-
name: "bazel //tools:lint_check"
105+
name: "./test_lint.sh"
106106
platform: ubuntu2004
107-
run_targets:
108-
- "//tools:lint_check"
107+
shell_commands:
108+
- "./test_lint.sh"
109109
test_rules_scala_jdk21:
110110
name: "./test_rules_scala with jdk21"
111111
platform: ubuntu2004

test_rules_scala.sh

+3-3
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@ test_dir=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )/test/shell
1313
runner=$(get_test_runner "${1:-local}")
1414

1515
"${test_dir}"/test_bzlmod_macros.sh
16-
$runner bazel build test/...
17-
#$runner bazel build "test/... --all_incompatible_changes"
18-
$runner bazel test test/...
16+
$runner bazel build "src/... test/..."
17+
#$runner bazel build "src/... test/... --all_incompatible_changes"
18+
$runner bazel test "src/... test/..."
1919
$runner bazel test third_party/...
2020
$runner bazel build "--extra_toolchains=//test/toolchains:high_level_transitive_deps_strict_deps_error -- test/..."
2121
$runner bazel build "--extra_toolchains=//scala:minimal_direct_source_deps -- test/..."

0 commit comments

Comments
 (0)