Skip to content

Comments

fix: resolve all PHPStan errors#113

Merged
deviantintegral merged 1 commit intomainfrom
claude/fix-phpstan-errors-QTu8x
Dec 19, 2025
Merged

fix: resolve all PHPStan errors#113
deviantintegral merged 1 commit intomainfrom
claude/fix-phpstan-errors-QTu8x

Conversation

@deviantintegral
Copy link
Owner

  • Add void return types to checkComposerVersion(), writeConstraint(), and all test methods (missingType.return)
  • Change PHPDoc @var union types (|) to intersection types (&) for mock objects to properly indicate the variable is both MockObject and the mocked interface (varTag.nativeType)
  • Remove dead code branch for Composer < 2.0 plugin API since the project requires composer-plugin-api ^2.0 (if.alwaysFalse, argument.type)
  • Simplify validate() closure return logic to remove always-true condition (booleanOr.rightAlwaysTrue)

- Add void return types to checkComposerVersion(), writeConstraint(),
  and all test methods (missingType.return)
- Change PHPDoc @var union types (|) to intersection types (&) for mock
  objects to properly indicate the variable is both MockObject and the
  mocked interface (varTag.nativeType)
- Remove dead code branch for Composer < 2.0 plugin API since the
  project requires composer-plugin-api ^2.0 (if.alwaysFalse, argument.type)
- Simplify validate() closure return logic to remove always-true
  condition (booleanOr.rightAlwaysTrue)
@deviantintegral deviantintegral merged commit 1c1ad2e into main Dec 19, 2025
7 checks passed
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.

2 participants