chore: fix remaining warnings for all targets

Author: NathanFlurry

.github/workflows/release.yaml

@@ -79,30 +79,30 @@ jobs:
       matrix:
         include:
           - platform: linux
-            runner: depot-ubuntu-24.04-4
+            runner: depot-ubuntu-24.04-8
             target: x86_64-unknown-linux-musl
             binary_ext: ""
             arch: x86_64
           # TODO: Add back when working
           # - platform: linux
-          #   runner: depot-ubuntu-24.04-arm-4
+          #   runner: depot-ubuntu-24.04-arm-8
           #   target: aarch64-unknown-linux-musl
           #   binary_ext: ""
           #   arch: aarch64
           - platform: windows
-            runner: depot-ubuntu-24.04-4
+            runner: depot-ubuntu-24.04-8
             target: x86_64-pc-windows-gnu
             binary_ext: ".exe"
             arch: x86_64
           - platform: macos
             # Use Linux instead of macOS builders since macOS does not support Docker
-            runner: depot-ubuntu-24.04-4
+            runner: depot-ubuntu-24.04-8
             target: x86_64-apple-darwin
             binary_ext: ""
             arch: x86_64
           - platform: macos
             # Use Linux instead of macOS builders since macOS does not support Docker
-            runner: depot-ubuntu-24.04-4
+            runner: depot-ubuntu-24.04-8
             target: aarch64-apple-darwin
             binary_ext: ""
             arch: aarch64
@@ -162,10 +162,10 @@ jobs:
       matrix:
         include:
           - platform: linux/arm64
-            runner: depot-ubuntu-24.04-arm-4
+            runner: depot-ubuntu-24.04-arm-8
             arch_suffix: -arm64
           - platform: linux/x86_64
-            runner: depot-ubuntu-24.04-4
+            runner: depot-ubuntu-24.04-8
             arch_suffix: -amd64
     runs-on: ${{ matrix.runner }}
     steps:
@@ -188,7 +188,7 @@ jobs:
           github_token: ${{ secrets.GITHUB_TOKEN}}
 
       - name: Build & Push (rivetkit/engine:full)
-        uses: docker/build-push-action@v4
+        uses: depot/build-push-action@v1
         with:
           context: .
           push: true
@@ -205,7 +205,7 @@ jobs:
           #   netrc=${{ runner.temp }}/netrc
 
       - name: Build & Push (rivetkit/engine:slim)
-        uses: docker/build-push-action@v4
+        uses: depot/build-push-action@v1
         with:
           context: .
           push: true

.github/workflows/rust.yml

@@ -2,27 +2,22 @@ name: Rust CI
 
 on:
   pull_request:
-    paths-ignore:
-      - 'frontend/**'
+    paths:
+      - '**/*.rs'
+      - '**/Cargo.toml'
+      - '**/Cargo.lock'
+      - '.github/workflows/rust.yml'
   push:
     branches:
       - main
-
-concurrency:
-  group: rust-${{ github.ref }}
-  cancel-in-progress: true
-
-permissions:
-  contents: read
-
-defaults:
-  run:
-    # Enable fail-fast behavior
-    shell: bash -e {0}
+    paths:
+      - '**/*.rs'
+      - '**/Cargo.toml'
+      - '**/Cargo.lock'
+      - '.github/workflows/rust.yml'
 
 env:
   # Disable incremental compilation for faster from-scratch builds
-  CARGO_INCREMENTAL: 0
   RUSTFLAGS: "--cfg tokio_unstable"
   CARGO_TERM_COLOR: always
 
@@ -43,23 +38,26 @@ jobs:
 
   # clippy:
   #   name: Clippy
-  #   runs-on: depot-ubuntu-24.04
+  #   runs-on: depot-ubuntu-24.04-8
   #   steps:
   #     - uses: actions/checkout@v4
-      
+
   #     - uses: actions-rust-lang/setup-rust-toolchain@v1
   #       with:
   #         toolchain: stable
   #         components: clippy
-      
+
   #     - uses: Swatinem/rust-cache@v2
-      
+  #       with:
+  #         shared-key: "rust-ci"
+  #         cache-on-failure: true
+
   #     - name: Run clippy
   #       run: cargo clippy --all-targets --all-features -- -W warnings
 
   check:
     name: Check
-    runs-on: depot-ubuntu-24.04
+    runs-on: depot-ubuntu-24.04-8
     steps:
       - uses: actions/checkout@v4
 
@@ -68,6 +66,9 @@ jobs:
           toolchain: stable
 
       - uses: Swatinem/rust-cache@v2
+        with:
+          shared-key: "rust-ci"
+          cache-on-failure: true
 
       - name: Check
         run: cargo check --all-targets --all-features
@@ -77,7 +78,7 @@ jobs:
 
   # test:
   #   name: Test
-  #   runs-on: depot-ubuntu-24.04
+  #   runs-on: depot-ubuntu-24.04-8
   #   steps:
   #     - uses: actions/checkout@v4
   #
@@ -86,6 +87,12 @@ jobs:
   #         toolchain: stable
   #
   #     - uses: Swatinem/rust-cache@v2
+  #       with:
+  #         # Share cache across all Rust jobs
+  #         shared-key: "rust-ci"
+  #         # Save cache even if the build fails
+  #         cache-on-failure: true
+  #         # Only save cache on main branch to avoid cache pollution from PRs
   #
   #     - name: Run tests
   #       run: cargo test --verbose