Skip to content

[DependencyInjection][Yaml] dump enums with the !php/enum tag #53680

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jan 30, 2024
Merged

[DependencyInjection][Yaml] dump enums with the !php/enum tag #53680

merged 1 commit into from
Jan 30, 2024

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jan 30, 2024

Q A
Branch? 7.1
Bug fix? no
New feature? yes
Deprecations? no
Issues Fix #53636
License MIT

@carsonbot carsonbot added this to the 7.1 milestone Jan 30, 2024
@carsonbot carsonbot changed the title [DependencyInjection] [Yaml] dump enums with the !php/enum tag [DependencyInjection][Yaml] dump enums with the !php/enum tag Jan 30, 2024
@xabbuh xabbuh mentioned this pull request Jan 30, 2024
Closed
@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit 4a49af8 into symfony:7.1 Jan 30, 2024
@xabbuh xabbuh deleted the issue-53636 branch January 30, 2024 14:06
@xabbuh xabbuh mentioned this pull request Mar 4, 2024
Closed
@fabpot fabpot mentioned this pull request May 2, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicolas-grekas nicolas-grekas nicolas-grekas approved these changes

@yceruto yceruto yceruto approved these changes

Assignees
No one assigned
Labels
DependencyInjection Feature Status: Reviewed Yaml
Projects
None yet
Milestone
7.1
Development

Successfully merging this pull request may close these issues.

[Yaml] Dumper for enums return !php/const instead of !php/enum
4 participants
@xabbuh @nicolas-grekas @yceruto @carsonbot