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
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
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
Thank you! We’ve been testing against master-SNAPSHOT all along but we’ve
decided we’ll try to take this to production (probably next week) to give the
new ASM stuff a good run.
(which is a bit slow for us – we normally take alphas to production pretty much
as they appear 😊 )
Sean Corfield --
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
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
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
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
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
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
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
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
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
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
14 matches
Mail list logo