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

PHP 8.1 support #5945

Closed
wants to merge 1 commit into from
Closed

PHP 8.1 support #5945

wants to merge 1 commit into from

Conversation

SpacePossum
Copy link
Contributor

@SpacePossum SpacePossum commented Sep 2, 2021

@SpacePossum SpacePossum marked this pull request as draft September 2, 2021 17:15
@coveralls
Copy link

coveralls commented Sep 2, 2021

Coverage Status

Coverage remained the same at 92.832% when pulling b0edbe0 on SpacePossum:master_81_never into ef44dce on FriendsOfPHP:master.

@SpacePossum SpacePossum mentioned this pull request Sep 3, 2021
@SpacePossum SpacePossum added topic/PHP8.1 Related to features available in PHP 8.1+ kind/feature labels Sep 3, 2021
@alexander-schranz
Copy link

alexander-schranz commented Sep 10, 2021

I tested PHP 8.1 and did got an unexepcted change which behaves different to php 8.0 or lower versions: #5993. I tested this branch and the master branch the issue there still exist. I sadly could not find out what change in PHP 8.1 does produce this issue.

@SpacePossum
Copy link
Contributor Author

The issue you see is the result of PHP8.1 tokenizing code differently than before, especially the & has changed for which we have no fix/support ready.

@GrahamCampbell
Copy link
Contributor

Some work on that was started here: #5863.

@SpacePossum SpacePossum closed this Dec 1, 2021
@SpacePossum SpacePossum deleted the master_81_never branch December 1, 2021 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature topic/PHP8.1 Related to features available in PHP 8.1+
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants