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

Fix various issues with templates and variables #153

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

PaulSD
Copy link

@PaulSD PaulSD commented Dec 23, 2024

See the individual commit messages for details.

Fixes #93, Closes #110, Closes #119, Closes #147, Closes #148
(This includes/supersedes #110 and #148, and eliminates the need for #119 and #147)

@PaulSD
Copy link
Author

PaulSD commented Dec 26, 2024

@ildar170975
Anything you can do to help move this along?

@ildar170975
Copy link
Collaborator

Hi Paul.
Please give me some time for this. I saw your PRs but not analysed them yet.

swiergot and others added 4 commits December 26, 2024 16:48
If multiple variables are defined then subsequent variables can
currently reference prior variables using `vars['name']` but not just
`name`.  This enables variables to reference prior variables by name,
like templates can.
@PaulSD
Copy link
Author

PaulSD commented Dec 26, 2024

(I merged #148 here, since it is necessary to fix the build issues.)

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.

entities parameter does not accept variable
4 participants