This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2019-11-09
Channels
- # announcements (7)
- # beginners (84)
- # calva (3)
- # cljs-dev (19)
- # cljsrn (1)
- # clojure (64)
- # clojure-dev (31)
- # clojure-italy (13)
- # clojure-nl (6)
- # clojure-spec (4)
- # clojure-uk (35)
- # clojurescript (4)
- # clojurex (86)
- # datomic (4)
- # fulcro (11)
- # graalvm (93)
- # joker (5)
- # nrepl (32)
- # off-topic (45)
- # pedestal (4)
- # reagent (3)
- # rewrite-clj (12)
- # sql (17)
- # tools-deps (4)
- # vim (28)
Is there a reason why zprint dependencies are not provided directly with cider-nrepl as they are with puget and fipp?
Puget and fipp are grandfathered in. They'd be removed if it wouldn't break the backwards compatibility. The goal is to make cider lighter.
I am having problems giving zprint
this option map over nREPL: {:record {:to-string? true}}
, because I don’t seem to be able to bencode booleans to anything else but 0/1, and zprint checks that it is a boolean… Is it possible to send a boolean proper over bencode? Or is there some other way I can achieve this? Feels like a problem I should have encountered before, but I haven’t…
It feels quite limiting to only be able to use what can survive the bencode conversion to and fro
Googling ":nrepl.middleware.print/options"
doesn't yield that many results, so I'm guessing this doesn't get used much other than to pass {:right-margin 70}
or similar around
Yeah, it is a bit surprising that it took me so long to run in to this limitation. A bit hard to see how it could be fixed though...
I think if we allowed that to be passed as a (edn) string (v.s. bencode map), that could be better?
guess we could even allow boolean in the bencode transport. Would just need to think about how to encode it
I just had a look around at how various other bencode implementations dealt with this. Nothing jumped out as an especially good options
I'm using cider.nrepl.pprint/zprint-pprint
, and providing the options via nrepl.middleware.print/options
.
then think it might be better to patch https://github.com/clojure-emacs/cider-nrepl/blob/7cdbe4d32b4893ab1d856b92cc802f96bb31f1f5/src/cider/nrepl/pprint.clj#L86 to coerce :to-string?
to boolean