Discards lexical ,

[arrdem]

It should be possible to use leading , to force indentation. cljfmt currently strips leading , leading to unexpected changes in indentation.

[arrdem]

This is due to the treatment of , as simple whitespace by rewrite-clj, currently being addressed over there in xsc/rewrite-clj#44

[arrdem]

This is due to the treatment of , as simple whitespace by rewrite-clj, currently being addressed over there in xsc/rewrite-clj#44

[arrdem]

[rewrite-clj "0.5.0"] brings :comma nodes which makes this more tractable. https://github.com/xsc/rewrite-clj/releases/tag/clojars-0.5.0

[arrdem]

[rewrite-clj "0.5.0"] brings :comma nodes which makes this more tractable. https://github.com/xsc/rewrite-clj/releases/tag/clojars-0.5.0

[MatthewDarling]

Looking at the linked examples, it looks like you want to keep commas when they're the first non-whitespace character on the line.

And anywhere else, removing them is fine?

[MatthewDarling]

Looking at the linked examples, it looks like you want to keep commas when they're the first non-whitespace character on the line.

And anywhere else, removing them is fine?

[arrdem]

So I use ,, as a line prefix in require/import forms since none of the indenters happen to understand or format prefix lists meaningfully and that trick happens to work in CIDER. Same thing for cond and other forms which have consecutive pairs that are not typically block indented when on different lines.

In general, I'm not sure that cljfmt should be removing commas when they occur.

[arrdem]

So I use ,, as a line prefix in require/import forms since none of the indenters happen to understand or format prefix lists meaningfully and that trick happens to work in CIDER. Same thing for cond and other forms which have consecutive pairs that are not typically block indented when on different lines.

In general, I'm not sure that cljfmt should be removing commas when they occur.