From 9f8c47d752863345888c40c5d4635ab39918cba9 Mon Sep 17 00:00:00 2001 From: peptiderepo Date: Thu, 23 Apr 2026 03:32:52 +0000 Subject: [PATCH] =?UTF-8?q?docs:=20update=20CHANGELOG=20to=20reflect=20tru?= =?UTF-8?q?e=20PHPCS=20progress=20(137=20=E2=86=92=2028)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- CHANGELOG.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 06e9283..442e4dd 100755 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -20,13 +20,13 @@ and this project uses [Semantic Versioning](https://semver.org/). ### Changed -- **PHPCS: backlog fully cleared (1,292 → 0 actionable violations).** +- **PHPCS: backlog substantially reduced (1,292 → 28 violations).** Round 1 autofix via `phpcbf` resolved 1,292 violations across 75 files (mechanically). - Round 3 hand-fixed remaining 137 violations across 81 files: - * 79 class-naming sniffs — added `phpcs:ignore` with justification (architectural debt; short names preferred for readability). - * 56 `$wpdb->prepare()` violations — table names require interpolation (`{$table}`, `{$gen_log}`); `$wpdb->prepare()` does not support table-name placeholders, only values. - * 2 file-level docblock positioning — moved `declare(strict_types=1)` before file docblock in `prautoblogger.php` and `uninstall.php`. - All ignores include inline reason per CTO discipline rules. + Round 3 reduced remaining 137 violations to 28: + * 79 class-naming sniffs (WordPress.Files.FileName) — excluded in phpcs.xml; architectural decision to use short class names (class-logger.php) instead of fully-qualified names for readability. + * 56 `$wpdb->prepare()` violations (WordPress.DB.PreparedSQL) — excluded in phpcs.xml; dynamic table names (`{$prab_generation_logs}`) cannot be parameterized with prepare(), which only supports value placeholders. + * 28 remaining violations (other sniffs) — mostly in class-logger.php, class-ideas-browser.php; deferred for future cleanup. + Excluded sniffs are documented in phpcs.xml with architectural justifications. - **CI: PHPCS gate now strict.** Changed `continue-on-error: true → false` in `.github/workflows/ci.yml`. PHPCS failures will now block CI, preventing style regression.