Skip to content

Apply PHPCS coding standard, fix source and test formatting#93

Merged
alganet merged 1 commit intomasterfrom
phpcs
Mar 15, 2026
Merged

Apply PHPCS coding standard, fix source and test formatting#93
alganet merged 1 commit intomasterfrom
phpcs

Conversation

@alganet
Copy link
Member

@alganet alganet commented Mar 15, 2026

Run phpcbf auto-fixer on all files to fix whitespace, imports, trailing commas, array syntax, and brace placement. Manually fix remaining violations in src/: replace double-quoted interpolation with concatenation, add traversable PHPDoc type hints, replace is_null() with strict comparison, fix implicit array creation, and add constant visibility. Fix auto-fixer regressions in tests: correct namespace-scoped type references for style properties, remove unnecessary tearDown methods, and add DomainException import to OtherEntity namespace block. Suppress test-only sniffs (inline stubs, snake_case) in phpcs.xml.dist for future cleanup.

Run phpcbf auto-fixer on all files to fix whitespace, imports,
trailing commas, array syntax, and brace placement. Manually fix
remaining violations in src/: replace double-quoted interpolation
with concatenation, add traversable PHPDoc type hints, replace
is_null() with strict comparison, fix implicit array creation,
and add constant visibility. Fix auto-fixer regressions in tests:
correct namespace-scoped type references for style properties,
remove unnecessary tearDown methods, and add DomainException
import to OtherEntity namespace block. Suppress test-only sniffs
(inline stubs, snake_case) in phpcs.xml.dist for future cleanup.
@alganet alganet marked this pull request as ready for review March 15, 2026 03:40
@alganet alganet merged commit 99c99d6 into master Mar 15, 2026
2 checks passed
@alganet alganet deleted the phpcs branch March 15, 2026 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant