-
Notifications
You must be signed in to change notification settings - Fork 8
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
Broken with Clojure 1.9.0-alpha11 #5
Comments
The obvious fix breaks on all previous versions. This is why I'm giving up on Clojure |
😢 |
I tried changing |
I created a PR at #6 with the fix. Hope that helps. |
+1 for applying the PR by puredanger. Until this is fixed midje won't work, see: |
I will fix this this evening. |
I ask a favor. If - if - you agree with me that the following error message is unacceptably unhelpful in this age of Elixir and Elm, please respond to https://groups.google.com/forum/?hl=en#!topic/clojure/mIlKaOiujlo With respect to @puredanger (who is a better, more patient person than I am) the idea that "spec error messages are going to be increasingly common for all of us to see and understand and I think it is worth taking the time to slow down and actually read them" is just not going to fly these days. Although I am ~90% certain to abandon Clojure for Elixir+Elm, I still think the idea of Lisp is great. If you disagree, it would be appropriate to say that too.
|
This is fixed in 5.2.3. However, the real issue, that Midje is broken under Clojure 1.9alpha11, isn't solved. Midje had a bad Users of suchwow are fine, but the tests won't work under Clojure 1.9alpha11. |
Midje 1.9.0-alpha5 is now compatible with Clojure 1.9-alpha11. |
Such.sequences has a bad ns form with
:require
which is missing:
.https://github.com/marick/suchwow/blob/master/src/such/sequences.clj#L2
The text was updated successfully, but these errors were encountered: