[CI] Move clang-format and buildifer steps before build and test (#5)

To allow early failures especially since builds can take a lot longer.
cruise-automation · Oct 15, 2023 · 0c24c34 · 0c24c34
1 parent d69bc22
commit 0c24c34
Showing 1 changed file with 16 additions and 16 deletions.
diff --git a/.github/workflows/bazelBuildAndTestTcp.yml b/.github/workflows/bazelBuildAndTestTcp.yml
@@ -57,22 +57,6 @@ jobs:
                      -t mlir-tcp:ci \
                      .
 
-    - name: Bazel build mlir-tcp
-      run: |
-        docker run --rm \
-                   -v "$(pwd)":"/opt/src/mlir-tcp" \
-                   -v "${HOME}/.cache/bazel":"/root/.cache/bazel" \
-                   mlir-tcp:ci \
-                   bazel build --config=clang_linux //:tcp-opt
-
-    - name: Bazel test mlir-tcp (lit tests)
-      run: |
-        docker run --rm \
-                   -v "$(pwd)":"/opt/src/mlir-tcp" \
-                   -v "${HOME}/.cache/bazel":"/root/.cache/bazel" \
-                   mlir-tcp:ci \
-                   bazel test --config=clang_linux //test/...
-
     - name: Verify clang-format was run (cpp lint)
       run: |
         docker run --rm \
@@ -97,6 +81,22 @@ jobs:
           exit 1
         fi
 
+    - name: Bazel build mlir-tcp
+      run: |
+        docker run --rm \
+                   -v "$(pwd)":"/opt/src/mlir-tcp" \
+                   -v "${HOME}/.cache/bazel":"/root/.cache/bazel" \
+                   mlir-tcp:ci \
+                   bazel build --config=clang_linux //:tcp-opt
+
+    - name: Bazel test mlir-tcp (lit tests, aot compile tests)
+      run: |
+        docker run --rm \
+                   -v "$(pwd)":"/opt/src/mlir-tcp" \
+                   -v "${HOME}/.cache/bazel":"/root/.cache/bazel" \
+                   mlir-tcp:ci \
+                   bazel test --config=clang_linux //test/...
+
     # Switch back bazel cache directory to user ownership
     # to allow GHA post-cache step to save cache without
     # permissions issue.