Skip to content

bug: MultilineWhitespaceBeforeSemicolonsFixer - fix for discovering multi line calls#6938

Merged
Wirone merged 3 commits intoPHP-CS-Fixer:masterfrom
werlos:fix_MultilineWhitespaceBeforeSemicolonsFixer
May 5, 2023
Merged

bug: MultilineWhitespaceBeforeSemicolonsFixer - fix for discovering multi line calls#6938
Wirone merged 3 commits intoPHP-CS-Fixer:masterfrom
werlos:fix_MultilineWhitespaceBeforeSemicolonsFixer

Conversation

@kubawerlos
Copy link
Copy Markdown
Member

@kubawerlos kubawerlos commented May 4, 2023

Fixes #6886

For this bugfix let's focus on detecting start of the "chained" call, and carry on the fact that for the fixer "chained" means actually "multiline" and discuss it separately to establish what should be changed by the fixer.

@kubawerlos kubawerlos changed the title bug: MultilineWhitespaceBeforeSemicolonsFixer - bug: MultilineWhitespaceBeforeSemicolonsFixer - fix for discovering multi line calls May 4, 2023
@kubawerlos kubawerlos marked this pull request as ready for review May 4, 2023 20:38
@Wirone Wirone merged commit 12792fa into PHP-CS-Fixer:master May 5, 2023
@Wirone
Copy link
Copy Markdown
Member

Wirone commented May 5, 2023

Thank you @kubawerlos 🍻

@kubawerlos kubawerlos deleted the fix_MultilineWhitespaceBeforeSemicolonsFixer branch May 5, 2023 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

multiline_whitespace_before_semicolons (new_line_for_chained_calls) not working in 3.16

2 participants