Skip to content

Upgrade PHPStan to level 8, fix type issues, apply Rector#36

Open
sylfabre wants to merge 2 commits intomainfrom
fix/phpstan-rector
Open

Upgrade PHPStan to level 8, fix type issues, apply Rector#36
sylfabre wants to merge 2 commits intomainfrom
fix/phpstan-rector

Conversation

@sylfabre
Copy link
Contributor

Summary

  • Removed level: 5 override from phpstan.neon.dist (now uses default level 8)
  • Fixed 34 PHPStan errors by adding type hints and fixing type safety
  • Applied Rector fix: made data provider static
  • 3 errors baselined (entity getId() on object, pre-existing checked exceptions)

Test plan

  • Rector clean
  • PHPStan 0 errors (3 baselined)
  • PHPUnit passes (31 tests, 65 assertions)

🤖 Generated with Claude Code

sylfabre and others added 2 commits February 13, 2026 15:54
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sonarqubecloud
Copy link

1 similar comment
@sonarqubecloud
Copy link

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