-
Notifications
You must be signed in to change notification settings - Fork 91
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
[shopsys] resolve symfony/translations deprecations #2487
Conversation
2cf6f55
to
60f0b5b
Compare
8e32c7b
to
92b7fd7
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.
Hi, nice job, I have just a few notes 😉
packages/framework/tests/Unit/Component/Translation/Resources/Controller.php
Show resolved
Hide resolved
92b7fd7
to
b250b85
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.
Thanks for the fixes! I have also tested it, no more translator deprecations, great!
b250b85
to
d79d793
Compare
- use trans() with count parameter instead
d79d793
to
b92bdec
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
symfony/translations