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
Non-Latin characters support #840
Conversation
--------- Co-authored-by: Maciej Skrzypkowski <maciej.skrzypkowski@satoricyber.com>
Pull Request Test Coverage Report for Build 4478961184
💛 - Coveralls |
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.
This approach seems fine to me. @alamb is there any pre-existing context about the use of ASCII-only that could be relevant?
I do not have any such context. Maybe @andygrove @nickolay @Dandandan or @maxcountryman would like to weigh in |
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.
Makes sense to me -- thank you @mskrzypkows and @ankrgyl for the review
For the MSSQL dialect I just didn’t need it personally. Figured it was better to leave it unimplemented, rather than produce a possibly buggy solution or read up on Unicode character classes in Rust. Thanks for figuring it out! |
* Non latin characters --------- Co-authored-by: Maciej Skrzypkowski <maciej.skrzypkowski@satoricyber.com> * Test for mysql --------- Co-authored-by: Maciej Skrzypkowski <maciej.skrzypkowski@satoricyber.com>
No description provided.