Various non-idiomatic whitespace from fipp.clojure about fipp

[brandonbloom]

Just to confirm: It looks like you are using fipp.clojure/pprint, not fipp.edn/pprint, right? I should probably document the former in the README.

I don't make any promise of clojure.pprint compatibility. That said, none of the changes you identified are intentional or problematic to fix as far as I can tell. The general philosophy of the code formatter is to be heuristically idiomatic. That is, idiomatic if its achievable without non-local analysis, and formatted in a not-totally-ugly way otherwise.

I think that https://github.com/brandonbloom/fipp/blob/master/src/fipp/clojure.cljc is pretty approachable. I'm reasonably happy with the output for my use case (mainly: macro debugging), but I'm open to improvements as long as they are relatively low risk. For example, you could probably fix the extra space at the end of the ns form by adding a when to

fipp/src/fipp/clojure.cljc

Line 121 in 07d138e

(visit p head) " " (visit p ns-sym) :line

. If you'd like to take a crack at fixing any of these, I'm happy to provide guidance. Please open individual issues or pull requests for each change.

Thanks!

[brandonbloom]

closing this for my own tracking purposes - if you have specific fixes/problems or whatever in the future that meet the design restrictions, please open new tickets.