-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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(rust) emoji supported in single quote strings #4156
fix(rust) emoji supported in single quote strings #4156
Conversation
Build Size ReportChanges to minified artifacts in 5 files changedTotal change +51 B View Changes
|
Build Size ReportChanges to minified artifacts in 3 files changedTotal change +49 B View Changes
|
Build Size ReportChanges to minified artifacts in 5 files changedTotal change +45 B View Changes
|
3c99ed7
to
e8a26b4
Compare
Build Size ReportChanges to minified artifacts in 3 files changedTotal change +49 B View Changes
|
I’m not quite familiar with grammar format here, but it turns out this change seems to be very breaking for a lot of Rust code: code example from Rust by Example |
11.11.0 is completely unusable like this, I guess this needs a revert and point release? |
Thanks for the link to the example test. Let me add that as a test case. I think I see what went wrong here. |
Changes
Resolves #3933. Uses begin/end to wrap strings instead of a regex that tries to match the whole string - which we do not recommend - and this is why. Now emoji work "for free" because they are just more text inside an existing string.
Checklist
CHANGES.md