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

[12.x] Supports PHP 8.4 #1799

Merged
merged 5 commits into from
Jan 13, 2025
Merged

[12.x] Supports PHP 8.4 #1799

merged 5 commits into from
Jan 13, 2025

Conversation

crynobone
Copy link
Member

@crynobone crynobone commented Nov 13, 2024


league/event is still on an outdated version and has no compatible version with league/oauth2-server 8. To remove all deprecations we need to upgrade league/oauth2-server to version 9 (covered in #1797

@kyryl-bogach
Copy link
Contributor

@crynobone league/oaut2-server was merged 🎉

@crynobone
Copy link
Member Author

@kyryl-bogach it is still pending a new tag/release, unfortunately.

Signed-off-by: Mior Muhammad Zaki <[email protected]>
@Sephster
Copy link
Contributor

Will tag it today

@Sephster
Copy link
Contributor

Tagged now. Sorry for the delay. Christmas is busy - hope you all have a good holiday period when it comes!

@calebdw
Copy link

calebdw commented Jan 2, 2025

Can this be merged? It looks like all the dependencies have been completed

@crynobone crynobone marked this pull request as ready for review January 13, 2025 01:49
@fragkp
Copy link

fragkp commented Jan 13, 2025

@taylorotwell Is it possible to review this? 🙏 Passport is the last stopper for us to upgrade to 8.4

@crynobone
Copy link
Member Author

This doesn't 100% solve PHP 8.4 as many of the dependencies are still outdated and we need to focus on #1797 to modernize Laravel Passport.

@taylorotwell taylorotwell merged commit b06a413 into 12.x Jan 13, 2025
12 checks passed
@taylorotwell taylorotwell deleted the php84 branch January 13, 2025 17:40
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.

6 participants