ci: random improvements (#238)

* ci: save Rust cache only on build jobs

* ci: use action for clippy to show in GitHub UI

* ci: remove pointless filters

Author: seqre

.github/workflows/rust.yml

@@ -2,11 +2,7 @@ name: Rust CI
 
 on:
   push:
-    paths-ignore:
-      - "**/*.md"
   pull_request:
-    paths-ignore:
-      - "**/*.md"
   schedule:
     - cron: "0 0 * * *"
 
@@ -79,9 +75,16 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Run clippy
-        run: cargo clippy --no-deps -- -D warnings
+        uses: giraffate/clippy-action@v1
+        with:
+          reporter: github-pr-check
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          clippy_flags: '--no-deps -- -D warnings'
+          fail_on_error: true
 
   external-deps:
     if: github.event_name == 'push' || github.event_name == 'schedule' || github.event.pull_request.head.repo.full_name != github.repository
@@ -100,6 +103,8 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Install cargo-nextest
         uses: taiki-e/install-action@v2
@@ -145,6 +150,8 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Run the external dependencies
         run: docker compose up -d
@@ -181,6 +188,8 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Run fmt
         run: cargo fmt --all -- --check
@@ -202,6 +211,8 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Run cargo-machete
         uses: bnjbvr/cargo-machete@v0.8.0
@@ -226,6 +237,8 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Run cargo check with minimal versions
         run: |
@@ -256,6 +269,8 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Run cargo check with every combination of features
         run: cargo hack check --feature-powerset --exclude-features db --no-dev-deps
@@ -283,6 +298,8 @@ jobs:
 
       - name: Cache Cargo registry
         uses: Swatinem/rust-cache@v2
+        with:
+          save-if: false
 
       - name: Miri setup
         run: cargo miri setup