search

philoskim / debux

A trace-based debugging library for Clojure and ClojureScript.
468 stars 19 forks source link

Feature request: nbb support #28

Open darky opened 2 years ago

darky commented 2 years ago

More info:

https://github.com/babashka/nbb/issues/171#issuecomment-1106847290

philoskim commented 2 years ago

I will review this issue and find a way through if it is possible.

Thank you for the report!

borkdude commented 1 year ago

@philoskim I didn't see the nbb request but if you want I can help with this.

I have a babashka-compatible fork here:

https://github.com/philoskim/debux/compare/master...borkdude:debux:master

The issue with the current debux is that it tries to load the CLJS analyzer which isn't compatible with bb (also not with nbb). In .cljc files this is easy to avoid using a :bb reader conditional.

In the debug.dbgn namespace I wasn't able to do this, but I can try to find a way to make that work with clj, cljs, bb and nbb, if you're interested:

https://github.com/philoskim/debux/compare/master...borkdude:debux:master

philoskim commented 1 year ago

@borkdude I couldn't find the way to solve this issue. So if you help with this, I will be very honored and appreciate it so much!

borkdude commented 1 year ago

Babashka support: https://github.com/philoskim/debux/pull/30