Hi, I'm upgrading clj-kondo and I'm experiencing unexpected behavior since this commit (bump to SCI 0.10.47) https://github.com/clj-kondo/clj-kondo/commit/e43c24186bd77c659357f2ed1f862f80077d0f6a
I have a requiring-resolve in a macro hook that is triggering this error: error: No context found in: sci.ctx-store/*ctx*. Please set it using sci.ctx-store/reset-ctx!. Should I remove the requiring-resolve or is this a bug? This is my macroexpand hook:
(defmacro reprod []
(requiring-resolve 'clojure.core/identity)
nil)
I can provide more details if needed.Hmm issue welcome
so you have a workaround right?
I haven't attempted it since I'm just copying code from the real macro namespaces to the exported hooks. The workaround would need to work in both or I'd need to find an approach, which I'd rather avoid both if I can at all.
ok so you automated copying the macro
the fix is easy btw, but I guess you need a new release...
I could just make a hotfix release soon-ish
To give you an idea of the urgency, I've been running the tests on this lib against clj-kondo RELEASE and they've been failing since the regression was introduced. So not urgent.
I was just getting around to it.
if you are running on the JVM I can give you a stable JVM version, it's pushed on every commit
it does contain "SNAPSHOT" but has a timestamp in it and won't change
yep I can use that!
fwiw if you're interested I can expose a script from my repo that takes the git sha of a clj-kondo commit and runs my regression tests over it. might be useful in CI on your end too.
sure! I'm already running regression tests in CI for clj-kondo btw
if you follow this CI output, eventually it prints a released JVM version you can use: https://app.circleci.com/jobs/github/clj-kondo/clj-kondo/30636
I still have to add a test for this, but I'm 99% sure this fixes it.
confirmed it resolves the regression for my lib, via jvm git dep.
thanks @borkdude!