Update to Clojure 1.11.1 about riemann HOT 5 CLOSED

Hey @brandonvin 👋

I feel like the clojure version change was not expected in the previous commit and that part was reverted.

As you can see here, CI did not pass on that previous commit:
https://github.com/riemann/riemann/actions/runs/6698749641

The error makes me feel that the testing framework of clojure has evolved and a test that rely on the old behavior needs tuning:

Testing: riemann.test-test
Running tests in: riemann.test-test
  Running test: #'riemann.test-test/only-one-tap-per-context
    FAIL
      Expected: (re-find #"Tap :foo \(.+?:\) already defined at :" err)
      Actual: (not (re-find #"Tap :foo \(.+?:\) already defined at :" Unexpected error macroexpanding riemann.test/tap at (/tmp/form-init8155462538598659347.clj:1:6476).))
  Finished test: #'riemann.test-test/only-one-tap-per-context

Fixing this should allow to update clojure. Can you give it a try and submit a PR?

from riemann.

Thanks @smortex! It might be as simple as updating the assertion, since Clojure now reports macroexpansion errors differently by wrapping them with a CompilerException. I've opened a PR: #1039.

from riemann.

I meant to come back to the blockers but never did so thanks @brandonvin! Awesome.

from riemann.

The PR is merged, thanks again for the pointers and getting the tests running!

I think it would be appropriate to keep this issue open, until the change is released.

Would it be possible to cut a release sometime soon?

from riemann.

Released in 0.3.11.

from riemann.