-
-
Notifications
You must be signed in to change notification settings - Fork 320
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
Fixed Issue #6929 Escape unicode when inspecting a Str #7411
Conversation
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.
Youll need to run cargo test -p test_mono and check in the changes, otherwise lgtm
@ayazhafiz Thanks for the info. Did it 👍 |
gah you may need to rerun it to solve the merge conflicts |
3e78929
to
a329168
Compare
@ayazhafiz I did it 👍 |
ab88f73
to
6e80488
Compare
@ayazhafiz is there anything else i have to do, that this can get merged? |
added AUTHORS improved comment
@ayazhafiz this is moving fast here... every time i look there are new merge conflicts... could you please approve again. |
Looks like this shold be ready to be merged... |
Sorry for your troubles @timonkrebs! |
replaces #7389 that automatically was closed when syncing the fork.