Skip to content
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

Handle unescaped '$' as last character in string literals #17

Merged
merged 3 commits into from
Apr 4, 2024

Conversation

brouwers-tiobe
Copy link

@brouwers-tiobe brouwers-tiobe commented Apr 3, 2024

In PowerShell, it is allowed to have an unescaped dollar sign in an expandable string literal (double quotes) in the following cases:

  • It is the last character in a string
  • It is the last character of a line in a multiline string

This caused lexical errors (see PR33917); this pull request is to recognize this syntax.

Also: changed definitions for ends of 'here strings' to not include newline, so they are reported on the correct line
@brouwers-tiobe brouwers-tiobe marked this pull request as ready for review April 3, 2024 08:31
@brouwers-tiobe brouwers-tiobe merged commit c4b5dd9 into tiobe_fixes Apr 4, 2024
3 checks passed
@tiobe tiobe deleted a comment from it-tiobe May 27, 2024
@tiobe tiobe deleted a comment from it-tiobe May 27, 2024
@tiobe tiobe deleted a comment from it-tiobe May 27, 2024
@tiobe tiobe deleted a comment from it-tiobe May 27, 2024
@tiobe tiobe deleted a comment from it-tiobe May 27, 2024
@tiobe tiobe deleted a comment from it-tiobe May 27, 2024
@it-tiobe
Copy link

it-tiobe commented May 27, 2024

Build artifacts

Tool Download
pmd pmd-0.1.0+270.zip
CPD CPD-0.1.0+270.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants