-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
(Replaced) PowerShell mode for CodeMirror #3910
Conversation
…ns recognition). New tests (large scripts). More cleanup.
…ntact you, but please let me know if you have issues with this).
corrected digit matching regex corrected spelling of 'identifier' in grammar dictionary notCharacterOrDash explicitly includes lowercase characters improved variable matching splatted variables now match bare '$' and '@' are errors, not variables moved single-quoted string processing into tokenSingleQuoteString incomplete strings are errors now empty strings are no longer errors added support for here-string interpolation with nesting support added highlighting for splatted vars removed arbitrary stream advancement prior to default error in tokenBase enabled folding braces
…ove\ into mode-powershell
@ashmind Can you squash the commits? Thanks. |
MT('keyword', "[keyword if]"); | ||
|
||
MT('call_builtin', "[builtin Get-ChildItem]"); | ||
})(); |
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 needs an additional newline afterwards.
@Dominator008 I thought about squashing them, but wouldn't that eliminate @benpmiller and @VapidWorx from the history? |
@ashmind Maybe squash into three / four commits? |
@Dominator008 OK I'll try that. |
@Dominator008 Please see #3911. |
This is a PowerShell mode for CodeMirror.
I've done it a while ago, so I don't remember all details, but the following should work well:
${}
And I've just added some tests (around 94% code coverage).
Let me know if there are any issues.