This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2020-05-21
Channels
- # announcements (3)
- # babashka (26)
- # beginners (45)
- # calva (10)
- # chlorine-clover (9)
- # cider (4)
- # clj-kondo (7)
- # cljs-dev (10)
- # clojure (95)
- # clojure-europe (33)
- # clojure-france (5)
- # clojure-nl (3)
- # clojure-spec (2)
- # clojure-sweden (3)
- # clojure-uk (33)
- # clojurescript (54)
- # conjure (101)
- # core-async (14)
- # cursive (1)
- # data-science (91)
- # datascript (3)
- # datomic (16)
- # figwheel (3)
- # figwheel-main (15)
- # fulcro (35)
- # ghostwheel (5)
- # graalvm (13)
- # helix (29)
- # java (5)
- # jobs (6)
- # kaocha (6)
- # leiningen (1)
- # meander (12)
- # off-topic (54)
- # pathom (13)
- # re-frame (16)
- # reagent (8)
- # reitit (4)
- # rum (3)
- # shadow-cljs (49)
- # spacemacs (2)
- # sql (5)
- # tools-deps (10)
- # xtdb (8)
- # yada (3)
@vaelatern, @dominicm - have merged Dominic's PR so that falsey args bind correctly, and also have https://github.com/juxt/crux/pull/888 raised to fix the issue where unused falsey args cause tuples not to be yielded. Will wait on an internal review for the latter, and then we should be good to go
This code doesn't look right to me, I think the timeout
argument should be passed along to await-tx
: https://github.com/juxt/crux/blob/master/crux-core/src/crux/node.clj#L152-L155
@U06D9RGQM yes, sorry this slipped through! Are you actively using the sync API still? Or were you just curious?
@U050V1N74 I can't make a PR as I can't print at moment and then sign the CLA, hence my drive-by post here
@U899JBRPF I'm not using the sync api, I was curious for something else in the code
@U06D9RGQM no problem, will pop the change through. thanks again 🙂