Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[JsonStreamer] Removenikic/php-parser dependency#60544

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@mtarld
Copy link
Contributor

QA
Branch?7.4
Bug fix?no
New feature?yes
Deprecations?no
Issues
LicenseMIT

Removenikic/php-parser dependency, and generate PHP code with strings.

/cc@nicolas-grekas

GromNaN and chalasr reacted with thumbs up emojinicolas-grekas reacted with hooray emojirvanlaak reacted with eyes emoji
@carsonbot

This comment was marked as off-topic.

@OskarStarkOskarStark changed the title[JsonStreamer] Remove "nikic/php-parser" dependency[JsonStreamer] Removenikic/php-parser dependencyMay 26, 2025
@mtarldmtarldforce-pushed thefeat/json-streamer-remove-php-parser branch fromd39189e to89a3011CompareMay 26, 2025 07:19
@mtarldmtarldforce-pushed thefeat/json-streamer-remove-php-parser branch from89a3011 toe1f2d33CompareMay 26, 2025 13:17
@mtarldmtarld requested a review fromsoyukaMay 26, 2025 13:18
@mtarldmtarldforce-pushed thefeat/json-streamer-remove-php-parser branch frome1f2d33 toe5930b3CompareMay 26, 2025 13:29
@fabpotfabpot modified the milestones:7.3,7.4May 26, 2025
@fabpot
Copy link
Member

Thank you@mtarld.

@fabpotfabpot merged commitc038498 intosymfony:7.4May 29, 2025
11 checks passed
@mtarldmtarld deleted the feat/json-streamer-remove-php-parser branchMay 29, 2025 07:01
This was referencedOct 27, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@OskarStarkOskarStarkOskarStark left review comments

@fabpotfabpotfabpot approved these changes

+1 more reviewer

@soyukasoyukasoyuka approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

7.4

Development

Successfully merging this pull request may close these issues.

5 participants

@mtarld@carsonbot@fabpot@OskarStark@soyuka
[8]ページ先頭
©2009-2025 Movatter.jp