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.2 support #1256

Closed
wants to merge 3 commits into from
Closed

PHP 8.2 support #1256

wants to merge 3 commits into from

Conversation

emteknetnz
Copy link

@emteknetnz emteknetnz commented Dec 13, 2022

Add the #[\AllowDynamicProperties] attribute to squash deprecation warning for dynamic properties e.g. $this->$name = $value; within __set() methods

Change deprecated string interpolation "${var}" to sprintf() - I did not use "$var" or "{$var}" syntax as that failed linting e.g. Squiz.Strings.DoubleQuoteUsage.ContainsVar e.g. Variable "$message" not allowed in double quoted

Adds PHP 8.2 to CI matrix

@spawnia
Copy link
Collaborator

spawnia commented Dec 13, 2022

I do not maintain v14 anymore, v15 will be released soon. You can already try the beta.

@spawnia spawnia closed this Dec 13, 2022
@emteknetnz
Copy link
Author

OK fair enough - do you know roughly when the stable for 15 will be released?

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

2 participants