From 3c87b408ccb207178bfd7303ec6215ab3de4ec97 Mon Sep 17 00:00:00 2001 From: Joe Date: Thu, 12 Mar 2026 21:02:31 -0500 Subject: [PATCH] ci: enable Clippy warnings as errors in linting step - Updated the Clippy command in both `ci.ps1` and `ci.sh` to include the `-D warnings` flag, ensuring that all warnings are treated as errors during the linting process. This change enhances code quality by enforcing stricter linting rules. --- scripts/ci.ps1 | 2 +- scripts/ci.sh | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/ci.ps1 b/scripts/ci.ps1 index ba2199bb..2f593ff3 100644 --- a/scripts/ci.ps1 +++ b/scripts/ci.ps1 @@ -63,7 +63,7 @@ Record-Result "Selene" $selene Write-Step 6 "Rust Linting (Clippy) - Auto-fix" cargo clippy -j $BuildThreads --fix --allow-dirty --allow-staged 2>&1 | Out-Null Write-Host "Verifying..." -ForegroundColor Yellow -cargo clippy -j $BuildThreads --all-targets --all-features 2>&1 +cargo clippy -j $BuildThreads --all-targets --all-features -- -D warnings 2>&1 $clippy = $LASTEXITCODE Record-Result "Clippy" $clippy diff --git a/scripts/ci.sh b/scripts/ci.sh index 2518d166..d67aee25 100755 --- a/scripts/ci.sh +++ b/scripts/ci.sh @@ -64,7 +64,7 @@ record "Selene" $selene_exit step 6 "Rust Linting (Clippy) - Auto-fix" cargo clippy -j $BUILD_THREADS --fix --allow-dirty --allow-staged >/dev/null 2>&1 echo "Verifying..." -cargo clippy -j $BUILD_THREADS --all-targets --all-features 2>&1 +cargo clippy -j $BUILD_THREADS --all-targets --all-features -- -D warnings 2>&1 clippy_exit=$? record "Clippy" $clippy_exit