feat: add Laravel 13 support - #10
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the package metadata and CI configuration to declare and validate compatibility with Laravel 13.
Changes:
- Extend Composer constraints to allow
illuminate/*v13 andorchestra/testbenchv11. - Expand the GitHub Actions test matrix to run against Laravel 13 (excluding PHP 8.2).
- Update README requirements to mention Laravel 13 (plus small formatting edits).
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| README.md | Documents Laravel 13 support in requirements and adjusts Inertia section formatting/examples. |
| composer.json | Allows Laravel/Illuminate v13 and Testbench v11 for development/testing. |
| .github/workflows/tests.yml | Adds Laravel 13 to the CI matrix and excludes the known-incompatible PHP 8.2 combo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Requirements | ||
|
|
||
| - PHP 8.2+ | ||
| - Laravel 11 or 12 | ||
| - Laravel 11 or 12 or 13 |
There was a problem hiding this comment.
The Requirements section now lists Laravel 13 support, but it still states "PHP 8.2+" without clarifying that Laravel 13 is not compatible with PHP 8.2 (the CI matrix explicitly excludes PHP 8.2 + Laravel 13). Consider updating the requirement wording to reflect the effective minimum PHP version per Laravel major (e.g., PHP 8.2+ for Laravel 11/12, PHP 8.3+ for Laravel 13) so users aren’t misled.
|
|
||
| - PHP 8.2+ | ||
| - Laravel 11 or 12 | ||
| - Laravel 11 or 12 or 13 |
There was a problem hiding this comment.
Minor wording: "Laravel 11 or 12 or 13" reads awkwardly in documentation. Consider rephrasing to "Laravel 11, 12, or 13" (or "Laravel 11–13") for clarity.
| - Laravel 11 or 12 or 13 | |
| - Laravel 11, 12, or 13 |
No description provided.