Skip to content

chore: Support Laravel 12 - #21

Merged
kofimokome merged 1 commit into
devfrom
feat/support-laravel-12
Jul 20, 2026
Merged

chore: Support Laravel 12#21
kofimokome merged 1 commit into
devfrom
feat/support-laravel-12

Conversation

@kofimokome

@kofimokome kofimokome commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Updates illuminate dependencies to support both Laravel 11 and 12.

Required to update Trakli to Laravel 12

@kofimokome kofimokome assigned nfebe and unassigned nfebe Jul 17, 2026
@sourceant

sourceant Bot commented Jul 17, 2026

Copy link
Copy Markdown

Code Review Summary

This PR successfully adds support for Laravel 12 while maintaining compatibility with Laravel 11. The changes are straightforward and follow established patterns for Laravel package maintenance.

🚀 Key Improvements

  • Broadened version constraints for illuminate/support, illuminate/console, illuminate/filesystem, and illuminate/database to include version 12.x.

💡 Minor Suggestions

  • To ensure the test suite remains compatible, consider updating orchestra/testbench to ^9.0|^10.0 in a follow-up or in the require-dev section, as Laravel 12 requires Testbench 10.
  • Since Laravel 11 and 12 both require PHP 8.2 or higher, you might consider bumping the minimum PHP version constraint from ^8.1 to ^8.2 in a future major release to align with the framework's requirements.

@sourceant sourceant Bot 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.

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@kofimokome
kofimokome requested a review from nfebe July 17, 2026 22:08
@kofimokome
kofimokome merged commit ad22066 into dev Jul 20, 2026
1 check passed
@kofimokome
kofimokome deleted the feat/support-laravel-12 branch July 20, 2026 09:41
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