From b6f97bf481c89d31df1d15ca62536239c72c4750 Mon Sep 17 00:00:00 2001
From: "hugh-hackman[bot]"
 <266376744+hugh-hackman[bot]@users.noreply.github.com>
Date: Thu, 19 Mar 2026 00:06:49 +0000
Subject: [PATCH] fix: remove || true from shellcheck step per QA review

---
 .github/workflows/pr-validation.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/pr-validation.yaml b/.github/workflows/pr-validation.yaml
index f893593..b1a5f2e 100644
--- a/.github/workflows/pr-validation.yaml
+++ b/.github/workflows/pr-validation.yaml
@@ -29,7 +29,7 @@ jobs:
           if ls .github/scripts/*.sh 1>/dev/null 2>&1; then
             for script in .github/scripts/*.sh; do
               echo "Checking ${script}..."
-              shellcheck --severity=warning "$script" || true
+              shellcheck --severity=warning "$script"
             done
           else
             echo "No shell scripts to check"