Skip to content

Conversation

@onlime
Copy link
Contributor

@onlime onlime commented Nov 26, 2025

Please add support for PHP 8.5

Drop Laravel 10 / PHP 8.1 support
@bilfeldt
Copy link
Owner

Thanks for the PR @onlime 👍 Why did you drop PHP 8.1 support, that should works just fine still?

@onlime
Copy link
Contributor Author

onlime commented Nov 27, 2025

Thanks for the PR @onlime 👍 Why did you drop PHP 8.1 support, that should works just fine still?

oh, I am just generally against supporting any versions that have reached EOL. If you want to keep on supporting it, go ahead, but PHP 8.1 is probably not used by anyone out there anymore. PHP 7.4 is a different story, but neither in the Laravel world.

@bilfeldt
Copy link
Owner

Since there is no obvious reason for dropping support for any old PHP version, then I will happily merge this PR without dropping previous version.

Please feel free to tag me once the PR is updated, otherwise I will come to it later.

Thanks.

@onlime
Copy link
Contributor Author

onlime commented Nov 27, 2025

@bilfeldt ok, PHP 8.1 / Laravel 10 support is back, but I had to drop prefer-lowest again from the run-tests matrix.

@onlime
Copy link
Contributor Author

onlime commented Nov 27, 2025

also re-added PHP 8.1 / Laravel 10 support in bilfeldt/laravel-request-logger#42

@bilfeldt bilfeldt merged commit 0a12cdd into bilfeldt:main Nov 28, 2025
20 checks passed
@bilfeldt
Copy link
Owner

Thanks @onlime 👍

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