Jeff Evans

Results 29 comments of Jeff Evans

The README.md already says `removed vars or removed arities`, but it also is checking for deprecation, ex: `metabase/driver.clj:401:1: warning: metabase.driver/supports? was deprecated.`

Since Jackson appears to be used under the covers, I suspect this is [feasible](https://fasterxml.github.io/jackson-core/javadoc/2.7/com/fasterxml/jackson/core/JsonParser.html#getTokenLocation()).

> I think the correct usage should be `Mockito.withSettings().useConstructor("hi", 1, c)`, would that work? In case you want to specify multiple, it would become `Mockito.withSettings().useConstructor("hi", 1, c1, c2)`. This: `Mockito.mock(A.class,...

Having the same issue here (5 years on). Temurin 17 JDK, OS X 12.3, IntelliJ Ultimate 2021.3.2, Cursive 1.12.1-2021.3

Where would the docstring go for that? I think you'd run into the [same issue](https://stackoverflow.com/a/39547599/375670) as in regular Clojure (i.e. there is no symbol to which to attach it).

> `defmethod` actually does create a symbol under the hood for each method, so the `:doc` metadata _could_ go on that Interesting. But how useful would it be? Isn't that...

Thank you @tha2015 , this is a fix we would very much like to see.

I think this change should also cover `must` because it calls `do-keypath-transform`. Output of new benchmarks from my machine: ``` Benchmark: set keypath and nthpath at index to NONE versus...

This should cover most of #219, although it doesn't do anything to address `srange` performance.

Adding a currently failing test to spark a discussion on how best to handle this scenario (i.e. creating a predicate function that correctly handles seeing only the beginning, but not...