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

Update PHP CS Fixer to allow using PHP 8.2 #40845

Closed
wants to merge 2 commits into from

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Oct 9, 2023

Summary

Update PHP CS Fixer to 3.34 to allow using PHP 8.0 (currently used 3.4 version was still for PHP 7.4 and did not allow using PHP 8.2).
Also updated the nextcloud/coding-standard to 1.1. That caused the bigger diff (running composer cs:fix).

Checklist

* Also update nextcloud/coding-style to 1.1.1

Signed-off-by: Ferdinand Thiessen <[email protected]>
… coding standards

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux added 3. to review Waiting for reviews php Pull requests that update Php code dependencies labels Oct 9, 2023
@nickvergessen
Copy link
Member

Closing as duplicate of #39271 (comment)

@nickvergessen nickvergessen deleted the deps/update-php-cs-fixer branch October 10, 2023 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews dependencies php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants