-
Notifications
You must be signed in to change notification settings - Fork 519
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
clarify global scope meaning for named templates #1662
clarify global scope meaning for named templates #1662
Conversation
Signed-off-by: Louis GERARD <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Terry Howe <[email protected]> Signed-off-by: lgerard-pass <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
but IANAM
@TerryHowe do you know if there are any active maintainers for this project? It looks like work has seriously stalled here. |
@yxxhero @karenhchu could you please chime in here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quick grammatical suggestion. otherwise its good to go
Co-authored-by: Andrew Block <[email protected]> Signed-off-by: Karen Chu <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for your contribution! 👍
I just got bitten by the fact that you cannot use the
$
sign to access the root context when using a named template.This should solve #1399