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: Expand allowed characters in keys #4044 #4045
base: main
Are you sure you want to change the base?
Conversation
src/languages/yaml.js
Outdated
{ begin: /\w[\w :()\./\-*@]*:(?=[ \t]|$)/ }, | ||
{ // double quoted keys - with brackets | ||
begin: /"\w[\w :()\./-]*":(?=[ \t]|$)/ }, | ||
begin: /"\w[\w :()\./\-*@]*":(?=[ \t]|$)/ }, | ||
{ // single quoted keys - with brackets | ||
begin: /'\w[\w :()\./-]*':(?=[ \t]|$)/ }, | ||
begin: /'\w[\w :()\./\-*@]*':(?=[ \t]|$)/ }, |
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.
Awesome!
Can we please keep the -
at the very end of the character group so we don't need the extra escape?
Also, could we get a test case added to the markup tests to show that this works as intended?
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.
@joshgoebel I moved the location of - and created a test file. When running the test, it looks like the picture below, is that right?
Changes
This modification extends the definition of the key to include spaces, colon, slash, hyphen, and parentheses. It also changes the key to include more character types while maintaining the pattern that ends in colon.
Checklist
CHANGES.md