Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Only interpret vars wrapped in braces #760

Merged
merged 2 commits into from
Feb 5, 2016
Merged

Conversation

dadgar
Copy link
Contributor

@dadgar dadgar commented Feb 5, 2016

Only interpret vars wrapped in braces: $node.class -> ${node.class}

dadgar added a commit that referenced this pull request Feb 5, 2016
Only interpret vars wrapped in braces
@dadgar dadgar merged commit 5b1c6f8 into master Feb 5, 2016
@dadgar dadgar deleted the f-var-interprelation branch February 5, 2016 02:21
@VladRassokhin
Copy link

BTW: Maybe it's time to use HIL? :)

@VladRassokhin
Copy link

It's good news that almost all HCL based config files would use unified interpolations syntax, simplifies IDE support.

@VladRassokhin
Copy link

Seems first if statement in resolveConstraintTarget should check for prefix ${, not $

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 29, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants