@reefersleep 1.0.658 is out! first release in over a month, it includes several hooks improvements. This should help to get your hook to run.
Hmmm yeah that's supposed to work too. I'll have a look to see what's missing.
In any case, the blame functionality is working as expected. Thanks a bunch, @plexus!
Cool!!! Would you be able to share the code?
Sure 🙂 I could make a repo that does only this, with code annotations and a README that explains how to work with it. Maybe others would find that handy.
In that case, did you make is so that :post-summary
is available from the REPL? That would remove what would otherwise be a small blemish in the ease of understanding how to work with it.
I thought so. If you're gonna put it in a separate repo then it makes sense to make it a plug-in. I can help with that.
Thanks! I’ll try it right away 😄
Should my tests.edn
be like this? I still can’t get :post-summary
output.
#kaocha/v1
{:tests [{:id :unit
:test-paths ["test"]
:ns-patterns [".*"]}]
:plugins [:kaocha.plugin/hooks]
:kaocha.hooks/post-summary [util.hooks/blame-report]}
that looks correct, let me try that
I tried just :post-summary
without the ns, that didn’t work either
you definitely need the ns
bugfix release incoming...
please try 1.0.663
, sorry for the hassle
Are you kidding, you’ve been a tremendous help 🙏☺️
So, :kaocha.hooks/post-summary
works when running lein test
now, but not in the REPL.
That’s great!
I’m in doubt about whether you wanted to enable it in the REPL, though?
When calling run
, that is.