search

brandonbloom / fipp

Fast Idiomatic Pretty Printer for Clojure
525 stars 44 forks source link

Make usage of commas optional #63

Closed jeluard closed 5 years ago

jeluard commented 5 years ago

Pretty-printing maps uses commas.

Would you consider a PR to make the usage of commas optional?

brandonbloom commented 5 years ago

Could you please elaborate on why you want this?

jeluard commented 5 years ago

I'm using fipp to pretty print some EDN and render it to a web page. For cosmetic reasons I'd rather not have some commas.

Does this use case makes sense for fipp?

brandonbloom commented 5 years ago

Yes and no. I’m not planning on adding output preferences of any kind to the built in pretty implementation. However, the implementation is very small and both the ednize and printing engine are reusable. I suggest you create your copy/paste and tweak to your heart’s content.

On Fri, Apr 12, 2019 at 10:58 AM Julien Eluard notifications@github.com wrote:

I'm using fipp to pretty print some EDN and render it to a web page. For cosmetic reasons I'd rather not have some commas.

Does this use case makes sense for fipp?

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/brandonbloom/fipp/issues/63#issuecomment-482666979, or mute the thread https://github.com/notifications/unsubscribe-auth/AAHRfKnNlq_p-fpZ_fAdcp-qhmF_bFyXks5vgMk-gaJpZM4csBWq .