2024-11-18 clerk | Clojure Slack Archive

clerk

Akiz 2024-11-18T09:27:20.453949Z

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!

mkvlr 2024-11-18T09:51:29.205189Z

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

Akiz 2024-11-18T11:53:24.077849Z

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

lread 2024-11-18T12:54:32.083609Z

Thanks @borkdude! I'll go ahead and cut a new release of rewrite-clj.

borkdude 2024-11-18T12:57:04.359129Z

🙏

lread 2024-11-18T13:29:22.567659Z

https://clojurians.slack.com/archives/C06MAR553/p1731936507492269

🙏 2

Akiz 2024-11-18T18:40:34.365259Z

Thanks everybody, it was fast!

Clojurians Log v2

clerk