8000 [ErrorHandler] support non-empty-string/non-empty-list when patching return types by xabbuh · Pull Request #59114 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[ErrorHandler] support non-empty-string/non-empty-list when patching return types #59114

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 stat 8000 ement. 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
Dec 7, 2024
Merged

[ErrorHandler] support non-empty-string/non-empty-list when patching return types #59114

merged 1 commit into from
Dec 7, 2024

Conversation

xabbuh
Copy link
Member
@xabbuh xabbuh commented Dec 7, 2024
Q A
Branch? 7.3
Bug fix? no
New feature? yes
Deprecations? no
Issues
License MIT

see the failures in #59075

@xabbuh xabbuh requested a review from yceruto as a code owner December 7, 2024 08:58
@carsonbot carsonbot added this to the 7.3 milestone Dec 7, 2024
@OskarStark OskarStark mentioned this pull request Dec 7, 2024
Merged
@derrabus
Copy link
Member
derrabus commented Dec 7, 2024

Thank you @xabbuh.

@derrabus derrabus merged commit b7ed0a4 into symfony:7.3 Dec 7, 2024
5 of 11 checks passed
@xabbuh xabbuh deleted the pr-59075 branch December 7, 2024 11:53
@fabpot fabpot mentioned this pull request May 2, 2025
Merged
Sign up for f 7E66 ree to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OskarStark OskarStark OskarStark approved these changes

@derrabus derrabus derrabus approved these changes

@alexandre-daubois alexandre-daubois alexandre-daubois approved these changes

@yceruto yceruto Awaiting requested review from yceruto yceruto is a code owner

Assignees
No one assigned
Labels
ErrorHandler Feature Status: Reviewed
Projects
None yet
Milestone
7.3
Development

Successfully merging this pull request may close these issues.
5 participants
@xabbuh @derrabus @OskarStark @alexandre-daubois @carsonbot
0