Skip to content

Conversation

@localheinz
Copy link
Contributor

@localheinz localheinz commented Sep 11, 2022

This pull request

  • enables the new_with_braces fixer
  • runs make coding-standards

Replaces #540.
Follows #559.

💁‍♂️ For reference, see https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v3.4.0/doc/rules/operator/new_with_braces.rst.

Co-authored-by: Andreas Möller <[email protected]>
Co-authored-by: Mathias Reker <[email protected]>
@localheinz localheinz changed the title Enhancement: Enable new_with_braces fixer Enhancement: Enable new_with_braces fixer Sep 11, 2022
Co-authored-by: Andreas Möller <[email protected]>
Co-authored-by: Mathias Reker <[email protected]>
@localheinz localheinz force-pushed the feature/new-with-braces branch from 7600661 to bfa5268 Compare September 11, 2022 21:02
@cmb69
Copy link
Member

cmb69 commented Sep 12, 2022

Thank you for the PR!

This is fine for me, but @kamil-tekiela was against that in #540, and maybe others don't like this either. So let's wait for more feedback.

@morrisonlevi
Copy link
Contributor

I'm also fine with this. Not sure how much feedback we'll get ^_^

@kamil-tekiela kamil-tekiela merged commit 73a9969 into php:master Aug 29, 2023
@localheinz localheinz deleted the feature/new-with-braces branch August 29, 2023 11:29
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.

4 participants