Skip to content

Support laravel 13#23

Open
parkourben99 wants to merge 1 commit intoaarondfrancis:mainfrom
parkourben99:main
Open

Support laravel 13#23
parkourben99 wants to merge 1 commit intoaarondfrancis:mainfrom
parkourben99:main

Conversation

@parkourben99
Copy link
Copy Markdown

@parkourben99 parkourben99 commented Mar 23, 2026

Summary by CodeRabbit

  • Chores
    • Updated package compatibility to support Laravel 13 alongside existing supported versions.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 23, 2026

📝 Walkthrough

Walkthrough

Updated composer.json to add Laravel 13 compatibility by expanding version constraints for illuminate/support, illuminate/filesystem, illuminate/console to include ^13.0, and orchestra/testbench to include ^11.0.

Changes

Cohort / File(s) Summary
Dependency Version Constraints
composer.json
Expanded version constraints for Laravel Illuminate packages and orchestra/testbench to support Laravel 13 and testbench 11 alongside existing version ranges.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Dependency chains grow long and wide,
Laravel thirteen joins the ride!
Version bumps and testbench too,
Compatibility—old and new!
Hop along to greater heights! 🌱

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly describes the main change: adding support for Laravel 13 by updating Composer dependency constraints.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@composer.json`:
- Around line 14-16: The composer.json currently allows illuminate/* up to ^13.0
but still requires PHP "^8.2", causing conflicts for Laravel 13; open
composer.json and change the PHP platform requirement from "php": "^8.2" to
"php": "^8.3" so it matches Laravel 13's minimum PHP version (locate the "php"
entry in composer.json and update its value).
- Line 21: The composer dev dependency constraints are incompatible:
"orchestra/testbench" at ^11.0 requires PHPUnit >=11.5.50 but "phpunit/phpunit"
is constrained to ^10.5|^11.0; update the PHPUnit constraint to match Testbench
11.x (for example replace ^10.5|^11.0 with ^10.5|^11.5.50|^12.5.8|^13.0.0) or
alternatively downgrade/remove the ^11.0 segment from "orchestra/testbench" so
the two requirements are satisfiable; edit composer.json to adjust the
"phpunit/phpunit" version string or the "orchestra/testbench" version
accordingly and run composer update to verify.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bc3a1929-4cd7-4964-8665-b42acd54cc7a

📥 Commits

Reviewing files that changed from the base of the PR and between ea1d3fe and 1dd79bc.

📒 Files selected for processing (1)
  • composer.json

Comment thread composer.json
Comment on lines +14 to +16
"illuminate/support": "^10.0|^11.0|^12.0|^13.0",
"illuminate/filesystem": "^10.0|^11.0|^12.0|^13.0",
"illuminate/console": "^10.0|^11.0|^12.0|^13.0"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🌐 Web query:

What is the official minimum PHP version required by Laravel 13.x (illuminate/support 13.x)?

💡 Result:

The official minimum PHP version required by Laravel 13.x (illuminate/support 13.x) is PHP 8.3.

Citations:


Update PHP constraint to support Laravel 13 requirement.

The package permits Laravel 13 (illuminate/*: ^13.0) but requires only PHP ^8.2. Laravel 13 requires PHP 8.3 as a minimum, creating an unresolvable conflict—installs targeting Laravel 13 on PHP 8.2 will fail. Change Line 13 to "php": "^8.3".

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@composer.json` around lines 14 - 16, The composer.json currently allows
illuminate/* up to ^13.0 but still requires PHP "^8.2", causing conflicts for
Laravel 13; open composer.json and change the PHP platform requirement from
"php": "^8.2" to "php": "^8.3" so it matches Laravel 13's minimum PHP version
(locate the "php" entry in composer.json and update its value).

Comment thread composer.json
"laravel/pint": "^1.0",
"mockery/mockery": "^1.6",
"orchestra/testbench": "^8.21|^9.2|^10.0",
"orchestra/testbench": "^8.21|^9.2|^10.0|^11.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🌐 Web query:

What PHPUnit versions are supported by orchestra/testbench 11.x, and which Laravel versions does it target?

💡 Result:

orchestra/testbench 11.x supports PHPUnit ^11.5.50, ^12.5.8, or ^13.0.0 and targets Laravel Framework ^13.0.0.

Citations:


🏁 Script executed:

# Search for the PHPUnit constraint in composer.json
rg "phpunit" composer.json -A 1 -B 1

Repository: aarondfrancis/airdrop

Length of output: 169


Testbench 11.0 requires PHPUnit ^11.5.50, not ^11.0.

The constraint phpunit/phpunit: ^10.5|^11.0 includes PHPUnit versions 11.0.0–11.5.49, which are incompatible with orchestra/testbench:^11.0. This creates an unsatisfiable require-dev combination. Either remove ^11.0 from testbench or update the PHPUnit constraint to ^10.5|^11.5.50|^12.5.8|^13.0.0 to match testbench 11.x requirements.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@composer.json` at line 21, The composer dev dependency constraints are
incompatible: "orchestra/testbench" at ^11.0 requires PHPUnit >=11.5.50 but
"phpunit/phpunit" is constrained to ^10.5|^11.0; update the PHPUnit constraint
to match Testbench 11.x (for example replace ^10.5|^11.0 with
^10.5|^11.5.50|^12.5.8|^13.0.0) or alternatively downgrade/remove the ^11.0
segment from "orchestra/testbench" so the two requirements are satisfiable; edit
composer.json to adjust the "phpunit/phpunit" version string or the
"orchestra/testbench" version accordingly and run composer update to verify.

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