Clarification: Leading and trailing commas are removed?

oakmac / standard-clojure-style-js

Standard Clojure Style in JavaScript

ISC License

84 stars 1 forks source link

Clarification: Leading and trailing commas are removed? #73

Closed shaunlebron closed 1 month ago

shaunlebron commented 2 months ago

Edit: When I saw this I was confused at why commas were being removed, but now I see that it’s a good idea to remove trailing commas because newlines are good enough separators for key-value pairs which the commas are mostly (only?) used for.

I also saw in our code that commas are used as ellipses ,,, as placeholders. One was on a line by itself holding a close-paren like ,,,). I don’t think it should be supported, but we should be aware of it.

oakmac commented 1 month ago

end-of-line commas get removed as if they are whitespace (Issue #104)
commas on the next line can "hold" parens to prevent slurping (Issue #60)