Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support for Unicode-aware regular expressions in Monaco Monarch language definitions #98447

Merged
merged 2 commits into from
Jun 1, 2020

Conversation

D-to-the-K
Copy link
Contributor

This PR fixes #98446.

It introduces a new flag in the IMonarchLanguage interface, giving developers the ability to use unicode-aware regular expressions in their Monaco Monarch language specifications.

A separate PR is needed to update Monaco Monarch's documentation:
https://github.com/microsoft/monaco-editor/blob/master/website/monarch.html

It's also important to let developers know that the unicode flag may break existing regular expressions (i.e., /\"/ works, but /\"/u fails).

For more details, see: https://mathiasbynens.be/notes/es6-unicode-regex

@msftclas
Copy link

msftclas commented May 23, 2020

CLA assistant check
All CLA requirements met.

@alexdima
Copy link
Member

alexdima commented Jun 1, 2020

Thank you!

@alexdima alexdima added this to the May 2020 milestone Jun 1, 2020
@alexdima alexdima merged commit ac74b6f into microsoft:master Jun 1, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jul 16, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Unicode-aware regular expressions in Monaco Monarch language definitions
3 participants