This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2020-09-04
Channels
- # announcements (1)
- # babashka (41)
- # beginners (124)
- # cider (8)
- # clj-kondo (25)
- # cljs-dev (40)
- # clojars (4)
- # clojure (92)
- # clojure-europe (35)
- # clojure-italy (3)
- # clojure-nl (4)
- # clojure-uk (91)
- # clojuredesign-podcast (8)
- # clojurescript (41)
- # code-reviews (1)
- # cursive (11)
- # data-science (1)
- # datascript (76)
- # datomic (45)
- # emacs (4)
- # etaoin (3)
- # events (1)
- # figwheel-main (2)
- # fulcro (9)
- # graalvm (1)
- # jackdaw (6)
- # jobs (1)
- # jobs-discuss (3)
- # kaocha (4)
- # malli (25)
- # off-topic (42)
- # pathom (4)
- # reitit (11)
- # releases (2)
- # reveal (1)
- # shadow-cljs (53)
- # sql (4)
- # tools-deps (190)
- # vrac (19)
- # xtdb (6)
so i'm trying to build master as well as the ignore-form branch, but it seems i'm doing something wrong. i'm using the jdk 11 20.1.0 version of graal on linux -- bash script/compile gives me this: https://pastebin.com/uJZNZKFF i noticed that the build.md doc now mentions something about using jdk12 to update resources: https://github.com/borkdude/clj-kondo/blob/master/doc/build.md#optional-steps is that something i need to do to build from source now?
ok -- gotta go out for a bit, but when i get back i'll try the steps and update the docs if i'm successful 🙂
@borkdude so i tried the ignore-form
branch.
the basic functionality seems to work :thumbsup:
i don't know if i did something wrong, but the following seemed to also affect linting:
(let [x 1]
{:clj-kondo/ignore [:redundant-let]}
(let [y 2]
(inc x y)
))
$ ./clj-kondo --lint ignore-form.clj
ignore-form.clj:4:5: error: clojure.core/inc is called with 2 args but expects 1
linting took 17ms, errors: 1, warnings: 0
note: i left off the discard marker #_
(let [x 1]
;;{:clj-kondo/ignore [:redundant-let]}
(let [y 2]
(inc x y)
))
$ ./clj-kondo --lint ignore-form.clj
ignore-form.clj:3:3: warning: Redundant let expression.
ignore-form.clj:4:5: error: clojure.core/inc is called with 2 args but expects 1
linting took 7ms, errors: 1, warnings: 1
oh hmm, no this is intended, since the map makes the let not be redundant anymore ;)
Input welcome on having multiple config dirs for shared config/hook code: https://github.com/borkdude/clj-kondo/issues/992