search

philoskim / debux

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

Add tap> support; add Babashka compatibility and remove ClojureScript dependency in Clojure #31

Closed gnl closed 10 months ago

gnl commented 10 months ago

I understand this is a bit of an omnibus PR, but I'm still hoping I can get it merged like this without having to pick up the tweezers to try and pull it apart. :)

Babashka compatibility is based on your babashka branch and the changes were closely related to removing the ClojureScript dependency in Clojure-only projects (it can sometimes cause very unpleasant dependency conflicts).

This required Clojure 1.10.0+, so I went ahead and removed future-spec, which was also breaking Babashka. Looking at the State of Clojure survey and the Clojure versions commonly in use, this seems acceptable, I hope you concur.

Support for tap> and the :simple option is needed for gnl/playback.

I haven't added tap> support to dbgt yet.

philoskim commented 10 months ago

Thanks for PR!

I will release the new version of debux as soon as possible if it doesn't have any problems after investigating and testing.

gnl commented 10 months ago

Fantastic, thank you, and also thanks again for Debux in general – it's been a godsend. 🙏