Remove redundant cargo check job from CI

The bare `cargo check` job only checked the lib/bin crates and was
strictly less thorough than the existing Test Suite and Lints jobs,
both of which compile all targets. The redundant job also introduced
a stale-cache risk: if its artifacts were reused from a prior run,
compile errors in source files could go undetected.

Removing it so that Test Suite (cargo test) and Lints
(cargo clippy --all-targets) are the authoritative compile checks.

Author: dduugg

.github/workflows/ci.yml

@@ -18,15 +18,6 @@ env:
   CARGO_TERM_COLOR: always
 
 jobs:
-  check:
-    name: Check
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout sources
-        uses: actions/checkout@v4
-
-      - name: Run cargo check
-        run: cargo check
   test:
     name: Test Suite
     runs-on: ubuntu-latest
@@ -58,7 +49,6 @@ jobs:
     needs:
       - test
       - lints
-      - check
     outputs:
       new_version: ${{ steps.check_for_version_changes.outputs.new_version }}
       changed: ${{ steps.check_for_version_changes.outputs.changed }}