Add React support and Laravel 13 compatibility - #10
Conversation
- spatie/laravel-permission ^8 for Laravel 13. Shared changes: - Laravel 13 / Inertia v3 / testbench 11 / Pest 4-5 compatible constraints, PHPStan 2 config and an empty PHPStan baseline. - New Laravilt cover in the new brand. - Pint formatting. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reachedNext included review available in 51 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change broadens supported development dependency versions and updates PHPStan configuration. It also reformats one Composer constraint without changing its allowed versions. ChangesDependency constraints
PHPStan configuration
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Merge Risk: ⚪ Minimal · up to The compatibility and PHPStan configuration changes are mergeable with normal checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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 |
With PHPUnit 12 (Pest 4/5), configured coverage reports make the test run exit 1 without running any tests when no coverage driver is installed, which failed CI. Coverage stays available on demand with pest --coverage. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
What changes
spatie/laravel-permission^8 for Laravel 13.Shared across all Laravilt packages: every package with a Vue frontend now ships a React 19 + TypeScript twin in
resources/react(same props, markup,data-*attributes and server contract, so panels behave the same on both stacks), plus Laravel 13 / Inertia v3 / testbench 11 / Pest 4-5 compatible constraints, PHPStan 2 config, a new cover in the new brand, and Pint formatting.Verification
peston Laravel 13 (testbench 11, Pest 5): greenpint --test: cleantsc --noEmit), andlaravilt:install --stack=reactbuilds and logs in end to end in headless Chrome🤖 Generated with Claude Code
Summary by CodeRabbit