Re: [ANN] Clojure 1.10.0-alpha7

2018-09-07 Thread Alex Miller
PR'ed here: https://github.com/pedestal/pedestal/pull/591 For now, you could peg to the prior version of org.clojure/core.specs.alpha dep, version 0.2.36, to avoid this. On Friday, September 7, 2018 at 8:34:03 AM UTC-5, Alex Miller wrote: > > Hi Mamun, this was already broken but the spec on n

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-07 Thread Alex Miller
Hi Mamun, this was already broken but the spec on ns import clauses was tightened to be able to discover it. In particular, this: (io.opentracing.log.Fields) is invalid - it should be: (io.opentracing.log Fields) That was silently doing nothing before (it was not being imported) but now i

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-07 Thread Mamun
After update to version "1.10.0-alpha7", I found pedestal/log is broken pedestal version 0.5.4. Here is details info in bellow (ns io.pedestal.log "Logging via slf4j. Each logging level is a macro: trace, debug, info, warn, and error. Each namespace gets its own Logger. Arguments are key-v

RE: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Sean Corfield
September 5, 2018 5:39:16 AM To: clojure@googlegroups.com Subject: [ANN] Clojure 1.10.0-alpha7 deps.edn dependency: org.clojure/clojure {:mvn/version "1.10.0-alpha7"} 1.10.0-alpha7 includes the following changes since 1.10.0-alpha6: * Update deps to latest spec.alpha (0.2.176) and

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Matching Socks
A linter is a good place for a warning that = is used with one apparent argument. Eastwood to the rescue! src/easter/core.clj:6:3: suspicious-expression: = called with 1 args. (= x) always returns true. Perhaps there are misplaced parentheses? Since = (and also < and > ) are handy in conjunc

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Peter Hull
On Thursday, 6 September 2018 19:11:14 UTC+1, Andy Fingerhut wrote: > > Doing a little bit more digging, I just did a Google search for: > > clojure "expects more than one argument" > > Nice one, thanks. I did a similar search but didn't find that - instead a repo called "venantius/ultra" which ha

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Andy Fingerhut
Doing a little bit more digging, I just did a Google search for: clojure "expects more than one argument" and found this code hit that may be relevant for CIDER users: https://github.com/clojure-emacs/cider-nrepl/blob/master/src/cider/nrepl/middleware/test/extensions.clj#L28 Andy On Thu, Sep 6

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Andy Fingerhut
Thanks for the response. Makes sense. I haven't used CIDER before, and don't plan to try it out just to learn if it has a feature that checks unit tests for problems. If anyone else knows whether CIDER has a feature to do that, I would be a little curious to learn more about it. Thanks, Andy O

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Peter Hull
On Thursday, 6 September 2018 15:09:15 UTC+1, Andy Fingerhut wrote: > > > If you have a few minutes to try out the tests in my repo to see if you > get different results than I did, I'd be curious to hear if your results > are different. I'd be more curious to learn of a way to write a unit tes

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Andy Fingerhut
Sorry, wrong link to the unit test that I tried, which should have been: https://github.com/jafingerhut/catch-bad-unit-test/blob/master/test/catchme/core_test.clj#L5-L7 On Thu, Sep 6, 2018 at 7:08 AM Andy Fingerhut wrote: > Peter: > > I do not know why you are seeing this change in behavior. I

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Andy Fingerhut
Peter: I do not know why you are seeing this change in behavior. In order to see if I could determine why, I created my own unit test based on your description, but when I run my incorrectly unit test with Clojure 1.10.0-alpha6 or 1.10.0-alpha7, I get the same results -- no failures, and no "erro

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Stuart Halloway
Hi Erik, That does seem like a good idea. Thanks! Stu On Thu, Sep 6, 2018 at 4:39 AM Erik Assum wrote: > Would it be an idea to include something like > > ``` > clj -Sdeps '{:deps {org.clojure/clojure {:mvn/version "1.10.0-alpha7"’ > ``` > > in such announcements so the lazy of us could ju

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Peter Hull
On Wednesday, 5 September 2018 13:39:36 UTC+1, stuart@gmail.com wrote: > > 1.10.0-alpha7 includes the following changes since 1.10.0-alpha6: > I was pleased to see that going from alpha 6 to 7 found an error in one of my tests. I had something like (is (= (x (func y and the new version re

Re: [ANN] Clojure 1.10.0-alpha7

2018-09-06 Thread Erik Assum
Would it be an idea to include something like ``` clj -Sdeps '{:deps {org.clojure/clojure {:mvn/version "1.10.0-alpha7"’ ``` in such announcements so the lazy of us could just copy/paste it into our shells to try it out? Erik. > On 5 Sep 2018, at 14:39, Stuart Halloway wrote: > > org.cl

[ANN] Clojure 1.10.0-alpha7

2018-09-05 Thread Stuart Halloway
deps.edn dependency: org.clojure/clojure {:mvn/version "1.10.0-alpha7"} 1.10.0-alpha7 includes the following changes since 1.10.0-alpha6: - Update deps to latest spec.alpha (0.2.176) and core.specs.alpha (0.2.44) - CLJ-2373 - categorize