Clojurians Log v2

@gfredericks The patch ended up being much smaller that I had thought https://gist.github.com/nwjsmith/edd1ce97b9e5334ac3cbc49a77ffb339 I'm trying to track down the other part of that ticket though. I can't seem to find any reader conditionals related to :/ in the tests. Did that already get fixed up?

@nwjsmith do you have the latest master?

🤦

I see it now

I've attached the patch to the ticket. Do I need to fiddle with the JIRA workflow of the ticket at all?

No -- you got both of the :/ things?

Yeah, both of the keyword-symbol-serialization-roundtrip and edn-roundtrips tests

and they both pass

cool, taking a look now

@nwjsmith looks great! will push as soon as the tests pass on my end and then close the ticket thanks again

No problem. Thanks for your help and let me know if there's anything else you'd like me to take a crack at

TCHECK-125 looks manageable, if you're interested

I'd wrap all the defmethods in an if that checks if clojure.test/report is a MultiFn or whatever that class is, and if it's not then just print a warning to *err*