Skip to content

Conversation

@swooningfish
Copy link
Contributor

Laravel 12 Dependencies

@u01jmg3
Copy link
Contributor

u01jmg3 commented Feb 24, 2025

@swooningfish: looks like the README could do with a new entry under Laravel Version Compatibility.

@swooningfish
Copy link
Contributor Author

@swooningfish: looks like the README could do with a new entry under Laravel Version Compatibility.

Thanks have updated.

- Removed ^11 references for all the packages to have the same Laravel version constraint.
+ Added back in illuminate/view and set the version constraint to ^12
@swooningfish
Copy link
Contributor Author

amened with pull request comments.

@u01jmg3
Copy link
Contributor

u01jmg3 commented Feb 25, 2025

"extra": {
"branch-alias": {
"dev-master": "11.x-dev"

Does the branch alias also need to be bumped?

[![Laravel 4.2|5.x|6|7|8|9|10|11](https://img.shields.io/badge/Laravel-4.2|5.x|6|7|8|9|10|11-orange.svg)](http://laravel.com)

And the badge

@swooningfish
Copy link
Contributor Author

"extra": {
"branch-alias": {
"dev-master": "11.x-dev"

Does the branch alias also need to be bumped?

[![Laravel 4.2|5.x|6|7|8|9|10|11](https://img.shields.io/badge/Laravel-4.2|5.x|6|7|8|9|10|11-orange.svg)](http://laravel.com)

And the badge

Thanks, have added in the badge for v12

Not sure about the branch alias as i think the branch needs to be available on the remote, but can amend if required.

@u01jmg3
Copy link
Contributor

u01jmg3 commented Feb 25, 2025

Not sure about the branch alias as i think the branch needs to be available on the remote, but can amend if required.

Sure, one for Yajra then, it was done last time anyway: c28eca3

@resslinger
Copy link

Any update?

@u01jmg3
Copy link
Contributor

u01jmg3 commented Feb 25, 2025

@resslinger: I would suggest giving @yajra some time to consider this PR. They're +8h (making it 10pm) and it's not even the middle of the week. 🏄🏻‍♂️

@Seb33300
Copy link
Contributor

Unfortunately, the build is still failing, this time because of Larastan 2 not compatible with Laravel 12:

image

Larastan needs to be upgraded to 3.1.0

"larastan/larastan": "^3.1.0",

@sonarqubecloud
Copy link

Copy link
Owner

@yajra yajra left a comment

Choose a reason for hiding this comment

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

Looks good to merge, will fix phpstan issues on another PR. Thanks!

@yajra yajra merged commit 86e0c92 into yajra:master Feb 26, 2025
5 of 9 checks passed
@rcerljenko
Copy link

Could you tag a new release please?

@yajra yajra changed the title Laravel v12 Compatibility feat: Laravel v12 Compatibility Feb 26, 2025
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.

7 participants