This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2022-02-19
Channels
- # announcements (1)
- # architecture (8)
- # babashka (8)
- # beginners (68)
- # biff (1)
- # calva (2)
- # clj-kondo (13)
- # cljs-dev (2)
- # clojure (71)
- # clojure-art (26)
- # clojure-europe (14)
- # clojure-nl (10)
- # clojure-uk (4)
- # clojurescript (96)
- # community-development (6)
- # conjure (1)
- # datalog (2)
- # emacs (6)
- # fulcro (20)
- # hugsql (7)
- # lsp (6)
- # nextjournal (13)
- # off-topic (7)
- # portal (1)
- # reagent (5)
- # reveal (8)
- # sci (50)
- # shadow-cljs (8)
- # spacemacs (2)
- # tools-deps (9)
- # xtdb (6)
Drafting p/try and p/catch for promesa.I've configured :lint-as
so that kitchen-async.promise/try
and catch match the clojure core counterparts but it seems like kondo is checking for catch
and not (p/catch)
which is resulting in that lint error. Is that fixable somewhere along the way or perhaps is this a non-starter?
@U04V15CAJ Thoughts on this ^ Want to know if it's worth solving or if it's a can of worms for tools like kondo
@U8WFYMFRU p/catch already exists in promesa. how would you re-use it for this macro?
In kitchen-async they introduce a separate name (which was going to be my strategy too) . The way they set it up was that there was p/catch for p/try and p/catch* for promise.catch. Was thinking on reversing that so promesa.catch* would be the counterpart to promesa.try. But it seems like getting p/try working and using lint-as might create more problems given that kondo is throwing an error that there is no catch
statement in there
So if it's not too much trouble in kondo (willing to help with the PR) then I'll continue with the promesa PR, but if it just creates more problems then it may not be worth implementing
I see so I can start out with using hooks in my userland config, then can include it in the promesa config when I'm ready for that upstream PR?
I started https://github.com/clj-kondo/clj-kondo/discussions/1593. All with an interest and/or opinions are most welcome.