Fork me on GitHub
#clj-kondo
<
2021-03-22
>
richiardiandrea16:03:18

Hi there, say I have with-db-transaction called this way jdbc/with-db-transaction [tx (:spec db) {:isolation :repeatable-read}] - how would I go in linting it so that it detects the last option map?

richiardiandrea16:03:33

I see this PR here https://github.com/clj-kondo/clj-kondo/pull/823/files I am checking if I can use it as well (using clojure.java.jdbc)

richiardiandrea16:03:25

ok never mind, it turns out I had a {clojure.java.jdbc/with-db-transaction clojure.core/let} and the above is built in so I just removed it

borkdude16:03:38

Does the current implementation of these macro analyzers not cover all usages?

richiardiandrea16:03:02

it does - see my above answer, I had miss-configured my config.edn

richiardiandrea16:03:13

thanks for checking in 😉

helios17:03:30

In this blogpost about our bootstrapped CI i show how we directly tap into Clj-kondo from a notebook for our CI linting https://nextjournal.com/blog/ci

borkdude17:03:30

@helios Nice incremental approach. > This is not something that couldn't have been achieved with a traditional CI and a bit of Bash scripting Since you are bringing up bash: also possible using babashka and the clj-kondo pod btw hehe

👍 3
borkdude17:03:19

But if you are running a JVM for testing anyway, yeah, why not run clj-kondo as part of it :)

borkdude17:03:03

@helios Maybe worth adding a link in ci-integration.md about this article

👍 3