More fixes related to #153#155
Conversation
Do you mean: |
|
Recreate the dev branch from master, then pull these changes into dev. Discard the old unreleased commits. |
Why it is needed? These commits do work (like from #109). |
|
I was trying to make the commit history more sane. But, OK, we can just add my changes. |
|
May be I understood you in a wrong way. |
|
Also, could you take a look at ALL workflow files? Imho they need to be updated. |
|
Ok, will look. |
Allow multiple templates anywhere in property value
Use lit 2.8.0 to match HA frontend version. Fixes iantrich#91, Closes iantrich#123, Closes iantrich#128, Closes iantrich#129, Closes iantrich#135
Fixes iantrich#93, Closes iantrich#119, Closes iantrich#147 (This eliminates the need for iantrich#119 and iantrich#147)
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.
This will drastically improve performance if any variable templates are expensive to eval.
|
Ok, I've fixed the authors, updated the workflow files, fixed a few more issues in the code. These commits don't apply cleanly on top of my prior commits in the dev branch, so we still need to re-create |
|
@PaulSD |
|
@PaulSD |
4 participants
After more testing, I found and fixed a bunch of issues with the commits in #153.
To avoid having broken commits in the history (in case anyone needs to bisect bugs), I've corrected the commits instead of adding more commits to fix them.
@ildar170975 Please discard the current
devbranch and re-create it with these commits.Built copy for anyone who wants to test this: https://github.com/PaulSD/config-template-card/releases/tag/1.3.6.2