Move clippy/fmt/deny checks to github

.github/workflows/ci.yml

@@ -30,6 +30,25 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-22.04
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          components: "clippy, rustfmt"
+      - uses: Swatinem/rust-cache@v2
+
+      # make sure all code has been formatted with rustfmt and linted with clippy
+      - name: rustfmt
+        run: cargo fmt -- --check --color always
+
+      # run clippy to verify we have no warnings
+      - run: cargo fetch
+      - name: cargo clippy
+        run: cargo clippy --all-targets --all-features -- -D warnings
+
   validate-proto-bufs:
     name: Validate proto bufs
     runs-on: ubuntu-22.04
@@ -53,3 +72,10 @@ jobs:
       - uses: Swatinem/rust-cache@v2
       - run: cargo fetch
       - run: cargo build
+
+  deny-check:
+    runs-on: ubuntu-22.04
+    steps:
+      - uses: actions/checkout@v4
+      - name: deny check
+        uses: EmbarkStudios/cargo-deny-action@v1

cloudbuild.yaml

@@ -38,34 +38,21 @@ steps:
     id: test-quilkin
     waitFor:
       - fetch-quilkin
-  - name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
-    dir: ./build
-    args:
-      - check-quilkin
-    id: check-quilkin
-    waitFor:
-      - fetch-quilkin
   - name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
     dir: ./build
     args:
       - test-examples
     id: test-examples
-    waitFor:
-      - fetch-quilkin
   - name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
     dir: ./build
     args:
       - test-docs
     id: test-docs
-    waitFor:
-      - fetch-quilkin
   - name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
     dir: ./build
     args:
       - build-image
     id: build
-    waitFor:
-      - fetch-quilkin
 
   #
   # Run the built images for 5 seconds in a few standard configurations, to test basic common scenarios