Skip to content

Conversation

@alongosz
Copy link
Member

@alongosz alongosz commented Oct 23, 2025

🎫 Issue n/a

Related PRs:

Description:

Yet another pre-release step for 2.2.0.

Bumped PHP CS Fixer fixed version constraint to the latest v3.89.0.

We had some deprecated rules. New fixer introduced @PER-CS2x0 set in place of @PER-CS2.0 one.
Also there were some pre-existing deprecations for v3.75.0:

- Rule "braces" is deprecated. Use "single_space_around_construct", "control_structure_braces", "control_structure_continuation_position", "declare_parentheses", "no_multiple_statements_per_line", "braces_position", "statement_indentation" and "no_extra_blank_lines" instead.
- Rule "native_function_type_declaration_casing" is deprecated. Use "native_type_declaration_casing" instead.
- Rule "new_with_braces" is deprecated. Use "new_with_parentheses" instead.
- Rule "no_unneeded_curly_braces" is deprecated. Use "no_unneeded_braces" instead.

It was quite complicated for the braces, so hopefully I got it right.

- Rule "braces" is deprecated. Use "single_space_around_construct", "control_structure_braces", "control_structure_continuation_position", "declare_parentheses", "no_multiple_statements_per_line", "braces_position", "statement_indentation" and "no_extra_blank_lines" instead.
- Rule "native_function_type_declaration_casing" is deprecated. Use "native_type_declaration_casing" instead.
- Rule "new_with_braces" is deprecated. Use "new_with_parentheses" instead.
- Rule "no_unneeded_curly_braces" is deprecated. Use "no_unneeded_braces" instead.
@alongosz alongosz force-pushed the bump-php-cs-fixer-to-v3.89.0 branch from d55fce7 to def867a Compare October 23, 2025 12:33
@sonarqubecloud
Copy link

Copy link

@konradoboza konradoboza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 but would be nice to see this "in action" to verify if there are rules that don't fit.

@alongosz
Copy link
Member Author

+1 but would be nice to see this "in action" to verify if there are rules that don't fit.

@konradoboza yeah, I'd like that too, but it's a bit difficult ATM, because of removal of FQCNs from the doc and multiline parameters. Applying 2.2.x-dev on core:main yields 2254 changed files due to #15, #16, and #13.

Maybe I'll try to cherry-pick this to 2.1.0 tag locally... :/

@konradoboza
Copy link

Maybe I'll try to cherry-pick this to 2.1.0 tag locally... :/

Didn't aim to suggest any additional work. We can iterate and decide once we start using bumped version.

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.

3 participants