Separate clippy/deny/fmt into separate job

googleforgames · Mar 1, 2024 · 0df0305 · 0df0305
1 parent 48dead7
commit 0df0305
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/build/Makefile b/build/Makefile
@@ -78,14 +78,16 @@ version:
 # Run all tests
 test: ensure-build-image test-quilkin test-examples test-docs
 
-# test only the quilkin crate
-test-quilkin: ensure-build-image
+check-quilkin: ensure-build-image
 	docker run --rm $(common_rust_args) \
 			--entrypoint=cargo $(BUILD_IMAGE_TAG) deny check
 	docker run --rm $(common_rust_args) \
 		--entrypoint=cargo $(BUILD_IMAGE_TAG) clippy --tests -- -D warnings
 	docker run --rm $(common_rust_args) \
 		--entrypoint=cargo $(BUILD_IMAGE_TAG) fmt -- --check
+
+# test only the quilkin crate
+test-quilkin: ensure-build-image
 	# --network=host because docker containers are not great at ipv6.
 	docker run --rm $(common_rust_args)  \
 			--network=host \

diff --git a/cloudbuild.yaml b/cloudbuild.yaml
@@ -31,6 +31,14 @@ steps:
     waitFor:
       - fetch-git-submodules
       - pull-build-image
+  - name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
+    dir: ./build
+    args:
+      - check-quilkin
+    id: check-quilkin
+    waitFor:
+      - fetch-git-submodules
+      - pull-build-image
   - name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
     dir: ./build
     args: