Skip to content

feat: add Laravel 13 support - #10

Merged
shavonn merged 1 commit into
mainfrom
feat/add-laravel-13-support
Mar 31, 2026
Merged

feat: add Laravel 13 support#10
shavonn merged 1 commit into
mainfrom
feat/add-laravel-13-support

Conversation

@shavonn

@shavonn shavonn commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 31, 2026 02:08
@shavonn
shavonn merged commit cfba22b into main Mar 31, 2026
21 checks passed
@shavonn
shavonn deleted the feat/add-laravel-13-support branch March 31, 2026 02:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 and orchestra/testbench v11.
  • 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.

Comment thread README.md
Comment on lines 28 to +31
## Requirements

- PHP 8.2+
- Laravel 11 or 12
- Laravel 11 or 12 or 13

Copilot AI Mar 31, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
Comment thread README.md

- PHP 8.2+
- Laravel 11 or 12
- Laravel 11 or 12 or 13

Copilot AI Mar 31, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
- Laravel 11 or 12 or 13
- Laravel 11, 12, or 13

Copilot uses AI. Check for mistakes.
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