Skip to content

Commit

Permalink
dev: bump da kondo (#302)
Browse files Browse the repository at this point in the history
Also, take advantage of clj-kondo's new ability to copy configs and
create cache at same time.
  • Loading branch information
lread authored Aug 3, 2024
1 parent 1d15fdd commit bf37e3d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 7 deletions.
2 changes: 1 addition & 1 deletion deps.edn
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@
;;
:lint-cache {:replace-paths ["src"]} ;; when building classpath we want to exclude resources
;; so we do not pick up our own clj-kondo config exports
:clj-kondo {:extra-deps {clj-kondo/clj-kondo {:mvn/version "2024.05.24"}}
:clj-kondo {:extra-deps {clj-kondo/clj-kondo {:mvn/version "2024.08.01"}}
:override-deps {org.clojure/clojure {:mvn/version "1.11.3"}}
:main-opts ["-m" "clj-kondo.main"]}

Expand Down
9 changes: 3 additions & 6 deletions script/lint_kondo.clj
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,8 @@
with-out-str
string/trim)
bb-cp (bbcp/get-classpath)]

(status/line :detail "- copying configs")
(shell/command "clojure -M:clj-kondo --skip-lint --copy-configs --lint" clj-cp bb-cp)
(status/line :detail "- creating cache")
(shell/command "clojure -M:clj-kondo --dependencies --parallel --lint" clj-cp bb-cp)))
(status/line :detail "- copying lib configs and creating cache")
(shell/command "clojure -M:clj-kondo --skip-lint --copy-configs --dependencies --lint" clj-cp bb-cp)))

(defn- check-cache [{:keys [rebuild-cache]}]
(status/line :head "clj-kondo: cache check")
Expand All @@ -53,7 +50,7 @@
(status/line :head "clj-kondo: linting")
(let [{:keys [exit]}
(shell/command {:continue true}
"clojure -M:clj-kondo --lint src test script deps.edn build.clj")]
"clojure -M:clj-kondo --parallel --lint src test script deps.edn build.clj")]
(cond
(= 2 exit) (status/die exit "clj-kondo found one or more lint errors")
(= 3 exit) (status/die exit "clj-kondo found one or more lint warnings")
Expand Down

0 comments on commit bf37e3d

Please sign in to comment.