Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(php): add support for PHP 8.4 #230

Open
wants to merge 14 commits into
base: stable
Choose a base branch
from

Conversation

pataar
Copy link

@pataar pataar commented Jan 27, 2025

Add tests for PHP 8.2, 8.3 and 8.4 + Laravel 9, 10 and 11.
Remove composer lock, as packages shouldn't have one.
Removed the league/pipeline dependency and replaced it with array_reduce.

@pataar
Copy link
Author

pataar commented Jan 27, 2025

image

@pataar
Copy link
Author

pataar commented Jan 28, 2025

@hisorange Any chance in checking this one out? It's currently blocking our use of your package.

@jhm-ciberman
Copy link

jhm-ciberman commented Jan 28, 2025

There is also #229, but the way it's implemented I think it conflicts with the "Standalone" mode. So I personally think this implementation is better since it does not depend on Laravel at all.

@hisorange ping to +1 the request

@jhm-ciberman
Copy link

@hisorange Sorry for the ping again, but could you merge and release this PR whenever you have time? This is the only thing blocking us to upgrade to PHP 8.4.

Thanks for your amazing work btw! 😁

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