Clojurians Log v2
Clojure programming
Channels
Apps
clerk 2024-11-18
Hi, i am having a problem with a reader, it fails on b//
It is this function: https://github.com/anteoas/broch/blob/main/src/broch/core.cljc#L105
Any ideas what to do (or should I create a ticket for that)? Thank you!
I don’t see b// there. Would appreciate an issue with a repro, thank you!
👍 1
borkdude 2024-11-18T11:18:12.725579Z
FWIW:
user=> (e/parse-string "b//")
b//
user=> (namespace (e/parse-string "b//"))
"b"
user=> (name (e/parse-string "b//"))
"/"borkdude 2024-11-18T11:18:17.916779Z
this is with edamame
borkdude 2024-11-18T11:20:59.010599Z
This notebook also works fine:
(ns scratch
(:require [nextjournal.clerk :as clerk]
[clojure.core :as b]))
b//borkdude 2024-11-18T11:22:58.393629Z
It looks like your exception is coming from rewrite-clj which depends on tools.reader. It would be interesting to see your deps tree. Perhaps it's pulling in an older version of tools.reader or so
borkdude 2024-11-18T11:27:00.530629Z
ah wait, I can reproduce this with the newest rewrite-clj. seems like a bug in rewrite-clj then (or tools/reader). I'll get to the bottom of this. you can probably work around this for now by downgrading rewrite-clj
🙏 1
Ah, makes sense, it worked before we updated rewrite-clj or related. Thank you!
borkdude 2024-11-18T11:53:52.592119Z
This fixes the issue: https://github.com/clj-commons/rewrite-clj/pull/324
👏 3
Thanks @borkdude! I'll go ahead and cut a new release of rewrite-clj.
borkdude 2024-11-18T12:57:04.359129Z
🙏
Thanks everybody, it was fast!