refactor: upgrade PHPStan to level 6 with type annotations#189
refactor: upgrade PHPStan to level 6 with type annotations#189danielhe4rt merged 5 commits into4.xfrom
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis pull request introduces comprehensive type safety improvements across the codebase. Changes include adding PHPDoc annotations and return type declarations to numerous methods, strengthening null-safety with null-safe operators and explicit null checks, updating Eloquent relationship type documentation to reflect actual pivot models, and refactoring some direct method chains to use typed local variables. The configuration updates PHPStan analysis level from 1 to 6 and adjusts analysis scope. Additionally, Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can approve the review once all CodeRabbit's comments are resolved.Enable the |
Summary
Changes
phpstan.neon: level 1 → level 6@property,@return, and@paramannotationsTesting
composer phpstanpassesSummary by CodeRabbit
Release Notes
Bug Fixes
Refactor
Chores