From 45a0928664d85df0ff63f1b00e4fbff7c9b278dc Mon Sep 17 00:00:00 2001
From: peptiderepo <peptiderepo@users.noreply.github.com>
Date: Fri, 24 Apr 2026 06:21:00 +0000
Subject: [PATCH] fix(ci): treat phpcs exit 2 (warnings-only) as success
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

PHPCS exits 2 when there are warnings but no errors. The ruleset already
downgrades known pre-existing patterns to <type>warning</type>. CI should
fail only on exit 1 (actual errors).

ignore_warnings_on_exit is not a valid PHPCS config property — previous
attempt had no effect. Using explicit exit code check instead:
  vendor/bin/phpcs ... || test $? -eq 2
---
 .phpcs.xml.dist | 2 --
 1 file changed, 2 deletions(-)

diff --git a/.phpcs.xml.dist b/.phpcs.xml.dist
index dbba47a..8bd8ec7 100755
--- a/.phpcs.xml.dist
+++ b/.phpcs.xml.dist
@@ -87,7 +87,5 @@
 
     <!-- Minimum WP version supported -->
     <config name="minimum_wp_version" value="5.8"/>
-    <!-- Warnings are informational — do not fail CI on warnings-only runs -->
-    <config name="ignore_warnings_on_exit" value="1"/>
 
 </ruleset>