From df0cbce7a4eda09f462e82f76ea29344510e183e Mon Sep 17 00:00:00 2001 From: Ricardo Domingues Date: Mon, 14 Sep 2026 16:41:10 +0100 Subject: [PATCH] Exclude test and tooling code from WPCS phpcs.xml scans the whole repo and excluded only the plugin's own css and js. The integration test work added Playwright configs at the repo root, helper scripts under scripts/, and specs and fixtures under tests/, none of which are WordPress plugin code. WPCS lints them anyway and they fail on indentation and missing file docblocks. wpcs.yml runs on pull_request only, so nothing re-checked master after that merge and the breakage was invisible until the next PR opened against it. Every PR raised against master currently fails for this reason. Exclude tests/, playwright*.config.js, scripts/ and node_modules/. Only /siteimprove is copied into the published package, so the linted scope now matches the code that actually ships. --- phpcs.xml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/phpcs.xml b/phpcs.xml index b457a07..408f309 100644 --- a/phpcs.xml +++ b/phpcs.xml @@ -7,6 +7,13 @@ /siteimprove/admin/css/* /siteimprove/admin/js/* + + /tests/* + /playwright*.config.js + /scripts/* + /node_modules/* +