Skip to content

[PHP 8.5] Add NestedToPipeOperatorRector#7577

Merged
TomasVotruba merged 4 commits intomainfrom
85-pipe
Oct 25, 2025
Merged

[PHP 8.5] Add NestedToPipeOperatorRector#7577
TomasVotruba merged 4 commits intomainfrom
85-pipe

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Oct 25, 2025

Rebased version of #7364 by @arshidkv12

@TomasVotruba TomasVotruba marked this pull request as draft October 25, 2025 14:55
@TomasVotruba TomasVotruba changed the title 85 pipe [PHP 8.5] Add NestedToPipeOperatorRector Oct 25, 2025
@TomasVotruba TomasVotruba marked this pull request as ready for review October 25, 2025 15:01
@TomasVotruba TomasVotruba merged commit 09ab43b into main Oct 25, 2025
53 checks passed
@TomasVotruba TomasVotruba deleted the 85-pipe branch October 25, 2025 15:06
@TomasVotruba
Copy link
Copy Markdown
Member Author

TomasVotruba commented Oct 25, 2025

Shipped and ready for first brave test runners 😄

This rule is not part of any set on purpose, as new syntax might be harder to get used to. Let's wait for the adoption in userland first.

Let's collect feedback now and implement fixes to make rule reliable 👍

@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants