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

Several php8.1 fixes #605

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

perdedora
Copy link
Member

The commit messages speaks for themself.

If you can, squash them.

@perdedora perdedora marked this pull request as ready for review July 22, 2023 02:48
@4chess
Copy link

4chess commented Sep 30, 2023

is all this going to be pushed to main?? Php 8.3 is coming so its good to at least have it 8.1 optimized!

@perdedora
Copy link
Member Author

Fixed conflicts

@4chess
Copy link

4chess commented Oct 8, 2023

Tested it for a while now on my production, not one php error! VERY nice work! I vouch for this, ppl should use it right away!

@Zankaria
Copy link

Any known reason as to why this wasn't merged?

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.

None yet

3 participants