8000 styles: Update to PHP-CS-Fixer 3.52.0 by theofidry · Pull Request #1929 · infection/infection · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

styles: Update to PHP-CS-Fixer 3.52.0 #1929

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Mar 19, 2024
Merged

styles: Update to PHP-CS-Fixer 3.52.0 #1929

merged 5 commits into from
Mar 19, 2024
+36 −23

Conversation

theofidry
Copy link
Member
@theofidry theofidry commented Mar 19, 2024 •
edited
Loading

A number of rules were disabled as they resulted in many changes. I will enable them one by one to make it easier to review.

maks-rafalko
maks-rafalko reviewed Mar 19, 2024
View reviewed changes
@theofidry theofidry merged commit 2885cf9 into infection:master Mar 19, 2024
@theofidry theofidry deleted the build/update-cs-fixer branch March 19, 2024 20:15
theofidry added a commit to theofidry/infection that referenced this pull request Mar 19, 2024
@theofidry
styles: Leverage the PHP class keyword
fed99d0 
Related to infection#1929.
@theofidry theofidry mentioned this pull request Mar 19, 2024
Merged
theofidry added a commit to theofidry/infection that referenced this pull request Mar 19, 2024
@theofidry
styles: Enable the operator line break PHP-CS-Fixer rule
84cd77a 
Related to infection#1929.
@theofidry theofidry mentioned this pull request Mar 19, 2024
Merged
theofidry added a commit to theofidry/infection that referenced this pull request Mar 19, 2024
@theofidry
styles: Leverage the PHP8 new str functions
9cd6f34 
Related to infection#1929.
@theofidry theofidry mentioned this pull request Mar 19, 2024
Merged
theofidry added a commit to theofidry/infection that referenced this pull request Mar 19, 2024
@theofidry
styles: Enable back the fully_qualified_strict_types PHP-CS-Fixer rule
6c04af7 
Related to infection#1929.
@theofidry theofidry mentioned this pull request Mar 19, 2024
Merged
theofidry added a commit to theofidry/infection that referenced this pull request Mar 19, 2024
@theofidry
styles: Enable back the blank_line_before_statement PHP-CS-Fixer rule
3b5dcb3 
Related to infection#1929.
@theofidry theofidry mentioned this pull request Mar 19, 2024
Merged
theofidry added a commit to theofidry/infection that referenced this pull request Mar 19, 2024
@theofidry
styles: Enable PHP-CS-Fixer array_indentation rule
0cd0892 
Related to infection#1929. This rule required more manual fixing.
@theofidry theofidry mentioned this pull request Mar 19, 2024
Merged
maks-rafalko pushed a commit that referenced this pull request Mar 19, 2024
@theofidry
styles: Leverage the PHP class keyword (#1931)
8c9cc85 
Related to #1929.
maks-rafalko pushed a commit that referenced this pull request Mar 19, 2024
@theofidry
styles: Enable PHP-CS-Fixer array_indentation rule (#1936)
e4dcc84 
* styles: Enable PHP-CS-Fixer array_indentation rule

Related to #1929. This rule required more manual fixing.

* manual fix

* more fix
maks-rafalko pushed a commit that referenced this pull request Mar 20, 2024
@theofidry @sanmai
styles: Enable back the fully_qualified_strict_types PHP-CS-Fixer rule (
efcb756 
#1934)

Related to #1929.

Co-authored-by: Alexey Kopytko <alexey@kopytko.com>
theofidry added a commit that referenced this pull request Mar 20, 2024
@theofidry
styles: Leverage the PHP8 new str functions (#1933)
f79348f 
Related to #1929.
theofidry added a commit that referenced this pull request Mar 20, 2024
@theofidry
styles: Enable the operator line break PHP-CS-Fixer rule (#1932)
eaebe44 
Related to #1929.
theofidry added a commit that referenced this pull request Mar 20, 2024
@theofidry
styles: Enable back the blank_line_before_statement PHP-CS-Fixer rule (…
4d8dc88 
…#1935)

Related to #1929.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sidz sidz sidz approved these changes

@maks-rafalko maks-rafalko maks-rafalko approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@theofidry @sidz @maks-rafalko
0