Conversation
|
This should be ready for review now. I've also added PHP 8.4 and 8.5 to the testing matrix. I had to add quite a few exclusions, based on the Laravel support policy. Because this package supports so many different combinations, I wonder if it might be worth refactoring to an |
|
Is the include list shorter? |
|
Yeah, it would look more like this: include:
# PHP 7.2
- { php-version: "7.2", laravel-version: "5.6.*" }
- { php-version: "7.2", laravel-version: "5.7.*" }
- { php-version: "7.2", laravel-version: "5.8.*" }
- { php-version: "7.2", laravel-version: "^6" }
- { php-version: "7.2", laravel-version: "^7" }
# PHP 7.3
- { php-version: "7.3", laravel-version: "5.6.*" }
- { php-version: "7.3", laravel-version: "5.7.*" }
- { php-version: "7.3", laravel-version: "5.8.*" }
- { php-version: "7.3", laravel-version: "^6" }
- { php-version: "7.3", laravel-version: "^7" }
- { php-version: "7.3", laravel-version: "^8" }
# PHP 7.4
- { php-version: "7.4", laravel-version: "5.6.*" }
- { php-version: "7.4", laravel-version: "5.7.*" }
- { php-version: "7.4", laravel-version: "5.8.*" }
- { php-version: "7.4", laravel-version: "^6" }
- { php-version: "7.4", laravel-version: "^7" }
- { php-version: "7.4", laravel-version: "^8" }
# PHP 8.0
- { php-version: "8.0", laravel-version: "^6" }
- { php-version: "8.0", laravel-version: "^7" }
- { php-version: "8.0", laravel-version: "^8" }
- { php-version: "8.0", laravel-version: "^9" }
# PHP 8.1
- { php-version: "8.1", laravel-version: "^8" }
- { php-version: "8.1", laravel-version: "^9" }
- { php-version: "8.1", laravel-version: "^10" }
# PHP 8.2
- { php-version: "8.2", laravel-version: "^8" }
- { php-version: "8.2", laravel-version: "^9" }
- { php-version: "8.2", laravel-version: "^10" }
- { php-version: "8.2", laravel-version: "^11" }
- { php-version: "8.2", laravel-version: "^12" }
# PHP 8.3
- { php-version: "8.3", laravel-version: "^8" }
- { php-version: "8.3", laravel-version: "^9" }
- { php-version: "8.3", laravel-version: "^10" }
- { php-version: "8.3", laravel-version: "^11" }
- { php-version: "8.3", laravel-version: "^12" }
- { php-version: "8.3", laravel-version: "^13" }
# PHP 8.4
- { php-version: "8.4", laravel-version: "^11" }
- { php-version: "8.4", laravel-version: "^12" }
- { php-version: "8.4", laravel-version: "^13" }
# PHP 8.5
- { php-version: "8.5", laravel-version: "^12" }
- { php-version: "8.5", laravel-version: "^13" } |
|
I would like to merge this as is and would appreciate a follow-up pull request. Fine with you? |
|
Sounds good! |
|
Thank you, released with https://github.com/laragraph/utils/releases/tag/v2.3.0. Please consider sponsoring. |
|
Thanks for the swift merge! It doesn't seem to be showing up on Packagist though. Any ideas? 🤔 |
|
Works now, forgot to mark it latest. |
|
Thanks for sorting it! |
Summary
This pull requests adds Laravel 13 support to the
laragraph/utilspackage. Laravel 13 is due to be released Early March.This PR also adds PHP 8.4 and 8.5 to the testing matrix.