Skip to content

Fix minor variable grammar bugs #22

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dolphinau
Copy link
Member

Create a new token to match the variable names only, without the $, @ or ${...}

@dolphinau dolphinau requested a review from citronneur June 12, 2025 15:34
@dolphinau dolphinau self-assigned this Jun 12, 2025
@dolphinau dolphinau added the enhancement New feature or request label Jun 12, 2025
@dolphinau
Copy link
Member Author

dolphinau commented Jun 13, 2025

New commits:
Adding variable_name doesn't look possible because we cannot break the variable token
However, I still fixed few bugs and added new corner cases:

  • $::
  • Remove : after reserved keywords
  • Add function and other PSDrive keywords
  • Add potential : in the variable name (ex: $local:test:test:test is valid but not $local:test::test:test)
  • Remove braced variable
  • Add escaped } with backticks for braced variables

@dolphinau dolphinau changed the title Add variable_name to match exactly the name of the variable if it exists Fix minor variable grammar bugs Jun 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant