Skip to content

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

Closed
pataar wants to merge 15 commits into
hisorange:stablefrom
pataar:remove_pipeline
Closed

chore(php): add support for PHP 8.4#230
pataar wants to merge 15 commits into
hisorange:stablefrom
pataar:remove_pipeline

Conversation

@pataar

@pataar pataar commented Jan 27, 2025

Copy link
Copy Markdown

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

@pataar

pataar commented Jan 27, 2025

Copy link
Copy Markdown
Author

image

@pataar

pataar commented Jan 28, 2025

Copy link
Copy Markdown
Author

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

@jhm-ciberman

jhm-ciberman commented Jan 28, 2025

Copy link
Copy Markdown

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
Copy Markdown

@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! 😁

@pataar

pataar commented Mar 28, 2025

Copy link
Copy Markdown
Author

@hisorange Friendly ping :)

@jhm-ciberman

Copy link
Copy Markdown

@hisorange Can you please publish this? This is holding us from upgrading to PHP 8.4

@pataar

pataar commented Mar 20, 2026

Copy link
Copy Markdown
Author

I created a fork which can be used as a drop-in replacement. It will be maintained. I also added Laravel 13 support: https://github.com/pataar/browser-detect

@pataar pataar closed this Mar 20, 2026
@pataar pataar deleted the remove_pipeline branch March 20, 2026 10:51
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