-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Document changes in the deprecation error handler #11323
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
eb6e5e2
to
8f8bce1
Compare
8f8bce1
to
884c063
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌🏻💥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! My PR is finally merged! 🎉 Here is a review reflecting the latest changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
@llaakkkk thanks a lot for this contribution! I'm happy to finally have it merged in Symfony Docs. Cheers! Note: GitHub displays this pull request as closed instead of merged ... that's because we changed the branch while merging (to merge it in 4.3 instead of master). But don't worry, it's merged. |
See symfony/symfony#29211
See symfony/symfony#28048
Closes #10701