search

trptcolin / reply

REPL-y: A fitter, happier, more productive REPL for Clojure.
Eclipse Public License 1.0
337 stars 44 forks source link

[Fix #204] Remove clojuredocs integration #209

Closed bbatsov closed 3 years ago

bbatsov commented 3 years ago

It has been broken for years, so let's just remove all this dead code.

Btw, what's is this export-definition abstraction about? In the absence of documentation for the function I couldn't quite figure it out.

trptcolin commented 3 years ago

Yikes, yep, sure is broken. I'm OK w/ this. 👍

It looks like maybe export-definition was about creating more of a seam to send code over to the client (https://github.com/trptcolin/reply/commit/0b208de7685808df4211fa4a4a57749607e0f0d6) but honestly I don't really remember. It does have the upside of letting us create real functions that are independently useful, instead of needing to embed the definitions in the big default-init-code thing.