Skip to content

[WIP] nikic/php-parser - #22

Draft
xepozz wants to merge 4 commits into
masterfrom
php-parser
Draft

[WIP] nikic/php-parser#22
xepozz wants to merge 4 commits into
masterfrom
php-parser

Conversation

@xepozz

Copy link
Copy Markdown
Member
QA
Is bugfix?✔️/❌
New feature?✔️/❌
Breaks BC?✔️/❌
Fixed issues

@samdark

Copy link
Copy Markdown
Member

@xepozz what's the plan on this one?

@xepozz

Copy link
Copy Markdown
MemberAuthor

It was for made for performance test comparasion. Current solution faster.

@xepozz

Copy link
Copy Markdown
MemberAuthor

But php-parser 5 is coming, need to compare with it also

@samdark

Copy link
Copy Markdown
Member

OK. Keeping it open for now.

@rustamwinrustamwin mentioned this pull request Sep 22, 2023
@samdark

Copy link
Copy Markdown
Member

@xepozz would you please re-compare it?

Sign up for freeto 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.

3 participants

@xepozz@samdark@StyleCIBot