Skip to content

Conversation

gharlan
Copy link
Member

@gharlan gharlan commented Jul 6, 2025

closes #8717

@gharlan gharlan changed the title NewExpressionParenthesesFixer - add to PHP84Migration set feat: NewExpressionParenthesesFixer - add to PHP84Migration set Jul 6, 2025
@gharlan gharlan changed the title feat: NewExpressionParenthesesFixer - add to PHP84Migration set feat: NewExpressionParenthesesFixer - add to PHP84MigrationSet Jul 6, 2025
@gharlan gharlan force-pushed the new-expression-parentheses-php84 branch from 0b894f8 to 424dec5 Compare July 8, 2025 07:37
@kubawerlos kubawerlos requested a review from keradus July 8, 2025 07:42
@coveralls
Copy link

coveralls commented Jul 8, 2025

Coverage Status

coverage: 94.78% (+0.01%) from 94.77%
when pulling e14a0b6 on gharlan:new-expression-parentheses-php84
into bb94482 on PHP-CS-Fixer:master.

@keradus keradus enabled auto-merge (squash) July 15, 2025 10:08
auto-merge was automatically disabled July 15, 2025 10:33

Head branch was pushed to by a user without write access

@@ -1,3 +1,5 @@
<?php

function sample(?string $str = null) {}

new DateTime()->getTimestamp();
Copy link
Member

Choose a reason for hiding this comment

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

🙇🏻

@keradus keradus merged commit 8092394 into PHP-CS-Fixer:master Jul 15, 2025
32 checks passed
@gharlan gharlan deleted the new-expression-parentheses-php84 branch July 15, 2025 11:32
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.

new_expression_parentheses - shall we add to 8.4 migration set ?
4 participants