Skip to content

Commit 79c1a4c

Browse files
committed
[ci] Run the format test before anything else in the rbe build
1 parent 6a49159 commit 79c1a4c

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

scripts/github-actions/ci-build.sh

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,19 @@ set -eufo pipefail
44
# We want to see what's going on
55
set -x
66

7+
# Fail the build if the format script needs to be re-run
8+
./scripts/format.sh
9+
git diff --exit-code
10+
711
# We want to use a pre-built Ruby version
812
echo 'RUBY_VERSION = "jruby-9.4.2.0"' >rb/ruby_version.bzl
913

1014
# The NPM repository rule wants to write to the HOME directory
1115
# but that's configured for the remote build machines, so run
1216
# that repository rule first so that the subsequent remote
13-
# build runs successfully.
14-
bazel query @npm//:all
17+
# build runs successfully. We don't care what the output is.
18+
bazel query @npm//:all >/dev/null
1519

1620
# Now run the tests. The engflow build uses pinned browsers
1721
# so this should be fine
18-
bazel test --config=remote --keep_going //java/...
22+
bazel test --config=remote-ci --keep_going //java/...

0 commit comments

Comments
 (0)