Conversation
Likely need to expose better hooks for this, but this seems good enough for now.
@kategengler - Thoughts on this one? |
|
That's amazing! |
|
Seconding amazing! For:
I'm fine with a major version bump. |
|
@rwjblue This does look amazing - any updates? |
|
@rwjblue how can I help get this across the finish line? |
|
Yes, same thoughts about amazing and how can we get this merged? Also, would it show coverage for the js side of helpers/components? |
|
@knownasilya this is likely on hold, due to the new spike. We'll be updating a lot in the coming weeks. |
|
This looks really nice. Would like to see this make it in 👍 |
|
@kategengler @rwjblue Are there any plans on this being completed soon? |
2c1fd17 to
755013a
Compare
|
The general idea here is still valid, just need to complete off the to do items. |
|
@rwjblue If this is on hold, is there any way i can contribute and get this merged ? |
|
Given that template only components are a prevalent thing, this would seem even more important |

This PR adds support for instrumentation of templates roughly the same as JS instrumentation (using a custom set of helpers and a template AST transform to instrument templates).
There are still some issues with this approach that we need to vet/review.
treeForto prevent app and addon tree from being emitted unlessCOVERAGE=true.htmlbars-inline-precompiler.Paired on this with @rondale-sc.