-
Notifications
You must be signed in to change notification settings - Fork 239
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
feat: implement basic template tags #1119
Conversation
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.
I realise this is a draft and I hope these comments are not premature - please feel free to ignore them if you are still changing things.
As you'll see, I am having second thoughts about what we chatted about yesterday, but only in the most milquetoast of ways - I'm still happy for you to go ahead doing it this way, just wanted to capture my thoughts in a comment.
c9adc48
to
e253528
Compare
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.
I'm delighted with this - thank you!
But... and I am really sorry about this... I think the new filtering stuff does need some unit-ish tests. But the actual code is really nice! I love that you got support into new/add, and it's all super clear to read.
a868fb4
to
b049eec
Compare
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.
Tests look good - thanks for adding them. I left one stylistic niggle but it's not a big deal - good to go!
Signed-off-by: Brian H <brian.hardock@fermyon.com>
b049eec
to
3659efa
Compare
This PR implements "template tags" to improve discoverability.
Signed-off-by: Brian H brian.hardock@fermyon.com