-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Fix logging with a string rejection #2311
Conversation
Nice. 👍 |
I tested it in a real project and that is what I needed |
@thewilkybarkid @Nyholm any news about this PR ? when it can be merged? |
Hello can we rebase here with master? |
52199d0
to
65e8c25
Compare
👍 |
This is good to be merged in its current state. Tests are red because we are waiting on #2495 |
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.
LGTM
Should this be backported to 6.5? |
8f6dd1b
to
41427fe
Compare
It could be backported if someone submits the PR. I would rather recommend people using 6.5 to upgrade to 7.0 =) |
Thank you @thewilkybarkid for fixing this. Also thank you everyone who took time to review and test it. |
Well, if we promise 6.x to be supported, then we kinda have to backport bugfixes, at least until everyone believes that 7.x works :P and until everyone stops using 5.x (yeah, I know) |
Promises aren't always rejected with an exception. I haven't changed
MessageFormatter
as that'd be a BC break.