Skip to content

stop-parsing symbol doesn't stop PowerShell highlighting #113

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

Closed
keith-hall opened this issue May 31, 2018 · 1 comment · Fixed by #118
Closed

stop-parsing symbol doesn't stop PowerShell highlighting #113

keith-hall opened this issue May 31, 2018 · 1 comment · Fixed by #118
Assignees
Labels

Comments

@keith-hall
Copy link
Contributor

Environment

  • Sublime 3176
  • Monokai

Issue Description

The PowerShell "stop-parsing" symbol is scoped correctly, but doesn't prevent the rest of the line being highlighted as PowerShell code.

Screenshots

image

Expected Behavior

The rest of the line following --% should be scoped as string.unquoted with no PowerShell constructs scoped like $VarName or /.

Code Samples

tf.exe status . /r --% /workspace:*;domain\$UserName

(code taken from https://stackoverflow.com/a/18928501/4473405)

@omniomi omniomi added the bug label May 31, 2018
@omniomi
Copy link
Contributor

omniomi commented May 31, 2018

Just need to add a capture for anything until the end of the line and make sure the stop-processing symbol is high in the definitions.

@omniomi omniomi self-assigned this Jun 9, 2018
omniomi added a commit to omniomi/EditorSyntax that referenced this issue Jun 9, 2018
@omniomi omniomi mentioned this issue Jun 9, 2018
TylerLeonhardt pushed a commit that referenced this issue Jun 11, 2018
* rescope statement separator per #114

* rescope everything after --% #113

* rescope string punctuation per #110

* include sigil in variable scope per #23

* rescope variable member per #107

* rescope variable sigil per #23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants