Merge remote-tracking branch 'origin/master' into noa/bump-rust-1.87

Author: coolreader18

.github/workflows/benchmarks.yml

@@ -177,7 +177,7 @@ jobs:
     name: run callgrind benchmarks
     # DON'T run on benchmarks-runner, using docker on a self-hosted runner has
     # been broken for 4 years: https://github.com/actions/runner/issues/434 .
-    # Fortunately, we can run on standard Github Actions infra because we don't care
+    # Fortunately, we can run on standard GitHub Actions infra because we don't care
     # about other stuff running on the machine!
     # runs-on: benchmarks-runner
     runs-on: ubuntu-latest
@@ -236,7 +236,7 @@ jobs:
           # summary PR" step). otherwise, we can use a fully shallow checkout
           fetch-depth: ${{ env.PR_NUMBER && 1 || 2 }}
 
-      - name: Unbork Github Actions state
+      - name: Unbork GitHub Actions state
         shell: bash
         run: |
           echo "Letting anybody touch our git repo, in order to avoid breaking other jobs"

.github/workflows/ci.yml

@@ -213,11 +213,20 @@ jobs:
       - uses: dsherret/rust-toolchain-file@v1
       - run: echo ::add-matcher::.github/workflows/rust_matcher.json
 
+      # Make sure the `Cargo.lock` file reflects the latest available versions.
+      # This is what users would end up with on a fresh module, so we want to
+      # catch any compile errors arising from a different transitive closure
+      # of dependencies than what is in the workspace lock file.
+      #
+      # For context see also: https://github.com/clockworklabs/SpacetimeDB/pull/2714
+      - name: Update dependencies
+        run: cargo update
+
       - name: Build module-test
         run: cargo run -p spacetimedb-cli -- build --project-path modules/module-test
 
       - name: Run bindgen tests
-        run: cargo test -p spacetimedb-cli
+        run: cargo test -p spacetimedb-codegen
 
   publish_checks:
     name: Check that packages are publishable