Refactor linter configuration and update Composer scripts for improved code quality checks.#551
Conversation
terabytesoftw
commented
Mar 4, 2026
| Q | A |
|---|---|
| Is bugfix? | ✔️ |
| New feature? | ❌ |
| Breaks BC? | ❌ |
| Tests pass? | ❌ |
…d code quality checks.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #551 +/- ##
=========================================
Coverage 18.36% 18.36%
Complexity 657 657
=========================================
Files 78 78
Lines 3496 3496
=========================================
Hits 642 642
Misses 2854 2854 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s local/CI linting setup by shifting PHPCS directory selection into phpcs.xml.dist, simplifying Composer scripts accordingly, and refreshing the PHPStan baseline to match the current static analysis output.
Changes:
- Update PHPCS configuration to explicitly scan
srcandtests, and simplify Composercs/cs-fixscripts to rely on the ruleset. - Remove the
directoriesinput from the reusable GitHub Actions linter workflow invocation. - Refresh
phpstan-baseline.neonentries and correct the heading indocs/internals.md.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
phpstan-baseline.neon |
Updates ignored PHPStan findings to match current analyzer output. |
phpcs.xml.dist |
Adds <file> entries for src and tests so PHPCS scope is defined in config. |
docs/internals.md |
Adjusts the top-level document heading. |
composer.json |
Simplifies cs / cs-fix scripts to run PHPCS/PHPCBF using the ruleset defaults. |
.github/workflows/linter.yml |
Removes explicit directories passed to the reusable linter workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.