This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2021-08-26
Channels
- # adventofcode (2)
- # announcements (7)
- # babashka (20)
- # beginners (77)
- # brompton (6)
- # calva (4)
- # clj-kondo (28)
- # cljdoc (2)
- # cljfx (10)
- # cljsrn (1)
- # cljtogether (1)
- # clojure (77)
- # clojure-europe (33)
- # clojure-gamedev (12)
- # clojure-uk (11)
- # clojurescript (95)
- # clojureverse-ops (4)
- # core-async (4)
- # core-logic (1)
- # cryogen (2)
- # cursive (14)
- # data-science (3)
- # datomic (47)
- # duct (1)
- # emacs (7)
- # fulcro (51)
- # gratitude (8)
- # helix (14)
- # hoplon (4)
- # improve-getting-started (60)
- # jobs (1)
- # jobs-discuss (4)
- # joker (11)
- # lsp (99)
- # meander (62)
- # membrane (5)
- # news-and-articles (3)
- # off-topic (64)
- # pathom (3)
- # polylith (11)
- # practicalli (7)
- # react (1)
- # reagent (8)
- # reveal (15)
- # shadow-cljs (78)
- # specter (7)
- # sql (16)
- # tools-build (1)
- # tools-deps (29)
- # workspaces (1)
- # xtdb (17)
Hi everybody, this might have been asked already, but why is this a linting error (using specter
). Did anybody run into this already?
What's the warning?
I might've just missed a couple things in my specter config that I added to the readme.
it's unresolved symbol
alright, I'll take a look at it
not yet, let me check that
@U5NCUG8NR is it the clj-kondo
readme?
No, it's in the specter readme. https://github.com/redplanetlabs/specter#clj-kondo
Oh cool, no I definitely haven't seen that
Yeah, that will fix a lot (if not all) of your unresolved symbol errors.
uhm it seems like they are still there
wait, blowing the cache might have solved it
Yeah, you have to blow the cache and run clj-kondo --dependencies --lint "$(clojure -Spath)"
when you do stuff like this.
It would be cool if specter could include this in its lib, but I guess README would do
yeah I was thinking the same
Yeah I asked nathan and the preference was to not include it in the artifact since not everyone uses kondo and it would increase the artifact size, even if only slightly.
I would be happy to PR this into the clj-kondo/config repo though.
@U5NCUG8NR I think we also have some instances of sp/richnav
like:
(def ^:private terminal2
(sp/richnav [afn]
(sp/select* [_this _vals _structure _next-fn]
(throw (Exception. "'terminal2' should only be used in multi-transform")))
(sp/transform* [_this vals structure _next-fn] (afn vals structure))))
yearh clj-kondo/config would be ok too. I think Nathan wasn't using clj-kondo at the time, but now he is, so perhaps his opinion has changed
I could run it by him again at some point.
Also @U0C8489U6 that's kinda out of scope for the minor config that I was making. That config is just meant to allow you to use all the vars from inside specter without them showing up as undefined. It's not meant to give you reasonable semantics for writing your own navs.
ok sounds good
I think I will just disable linting for that then for now
Although if nathan is interested in adding clj-kondo support to specter, I could consider writing hooks for the macros which would support custom navigators past not showing up as undeclared.
Which even if he isn't interested, that'd be a significant effort, so I'd want to at least pull it on clj-kondo/config
Thank you for your work on that Joshua, it really helped here