XKit-Rewritten icon indicating copy to clipboard operation
XKit-Rewritten copied to clipboard

Decide on trailing comma style

Open marcustyphoon opened this issue 2 months ago • 3 comments

Concept

As noted in https://github.com/neostandard/neostandard/issues/79, neostandard didn't come to a decision on trailing comma style, so inconsistency is currently allowed.

I personally am a big fan of multiline comma dangle, but that would be a change from standard. (I personally don't mind re-linting PRs; if I wasn't the author of almost every outstanding PR, I'd say this would be a big drawback.)

(biomejs defaults to trailing commas; while I don't really have an opinion on whether it's the new standard or not—pun not intended—it's at least one of the options.)

marcustyphoon avatar Oct 28 '25 20:10 marcustyphoon

I like trailing commas. If you want to do it, do it.

AprilSylph avatar Oct 28 '25 20:10 AprilSylph

Hm... while I don't mind re-linting PRs, merge conflicts would also be a thing.

Might hold off on this a bit as I feel there's a possibility our PR count could drop in the... what's between near future and far future? Middle future?

marcustyphoon avatar Oct 29 '25 05:10 marcustyphoon

Something we could do in the meantime is set up a lint rule for it, but set the severity to warn, so that we migrate to it gradually as part of natural code churn?

AprilSylph avatar Oct 29 '25 11:10 AprilSylph