CI: Run zebra-state Clippy separately and allow clippy::useless_conversion

Author: dmidem

.github/workflows/ci-basic.yml

@@ -43,19 +43,25 @@ jobs:
         run: cargo fmt -- --check
       - name: Run clippy
         run: |
-          # Keep clippy strict for most crates, but avoid churn from toolchain-specific lints
-          # in upstream Zebra code that started firing after the Rust 1.85 toolchain bump.
-          cargo clippy \
-            --workspace --no-deps --all-features --all-targets --locked \
-            --exclude zebra-chain \
-            -- -D warnings
-
-          cargo clippy \
-            -p zebra-chain --no-deps --all-features --all-targets --locked \
-            -- -D warnings \
-               -A elided_named_lifetimes \
-               -A clippy::needless_lifetimes \
-               -A clippy::unnecessary_lazy_evaluations
+          # After upgrading from Rust 1.82 to 1.85.1, Clippy started failing on new lints in upstream
+          # code we don’t want to patch here (and may already be fixed upstream). To avoid carrying
+          # fixes and merge conflicts, we allow these lints only for the affected crates for now.
+          # TODO: remove this comment and these Clippy exceptions after syncing with updated upstream.
+          cargo clippy --workspace --no-deps --all-features --all-targets --locked \
+            --exclude zebra-chain --exclude zebra-state -- -D warnings
+
+          # relaxed only for zebra-chain (your previous three)
+          cargo clippy -p zebra-chain --all-features --all-targets --locked -- \
+            -D warnings \
+            -A elided_named_lifetimes \
+            -A clippy::needless_lifetimes \
+            -A clippy::unnecessary_lazy_evaluations
+
+          # relaxed only for zebra-state (new)
+          cargo clippy -p zebra-state --all-features --all-targets --locked -- \
+            -D warnings \
+            -A clippy::useless_conversion
+
       - name: Show system resource summary
         run: |
           df -h