Skip to content

[Php81] Fix first class callable should not have trailing comma on FunctionLikeToFirstClassCallableRector#7551

Merged
samsonasik merged 2 commits intomainfrom
fix-trailing
Oct 23, 2025
Merged

[Php81] Fix first class callable should not have trailing comma on FunctionLikeToFirstClassCallableRector#7551
samsonasik merged 2 commits intomainfrom
fix-trailing

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@samsonasik samsonasik commented Oct 23, 2025

@samsonasik
Copy link
Copy Markdown
Member Author

Fixed 🎉 /cc @billypoke

@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit b2667d6 into main Oct 23, 2025
53 checks passed
@samsonasik samsonasik deleted the fix-trailing branch October 23, 2025 16:56
@TomasVotruba
Copy link
Copy Markdown
Member

I wonder if this should be handled by php-parser itself. Maybe better than hack around here

@samsonasik
Copy link
Copy Markdown
Member Author

Yes, I will try if I can do that

@samsonasik
Copy link
Copy Markdown
Member Author

I created test PR to nikic/php-parser for this case:

@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

Development

Successfully merging this pull request may close these issues.

Trailing Comma is preserved with FunctionLikeToFirstClassCallableRector resulting in ParseError

2 participants