Skip to content
This repository was archived by the owner on Mar 25, 2025. It is now read-only.

Added isName function and added '$' to test #124

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sampaioletti
Copy link

Fixes #118

@rohitkeshwani07
Copy link

I have a similar use case but don't u think it should be configurable?

@sampaioletti
Copy link
Author

Sure I think thats an idea, are you thinking ability to pass in whatever runes you would like or just ability to enable/disable '$'

@rohitkeshwani07
Copy link

rohitkeshwani07 commented Apr 30, 2019

Whatever is between [ and ] is treated as variable right now. What if these 2 can be configured??

Prefix and Suffix to find the variable?

For our usecase it will be $ and space

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

Successfully merging this pull request may close these issues.

Allow '$' prefix for variable and function names
3 participants