Prettier configuration changed #160
Conversation
package.json
Outdated
| @@ -10,7 +10,7 @@ | |||
| "typescript": "^5.2.2" | |||
| }, | |||
| "prettier": { | |||
| "trailingComma": "all", | |||
| "trailingComma": "none", | |||
There was a problem hiding this comment.
Is there a way to only remove the trailing commas for functions, but not on objects or arrays? I would like to minimize the impact on the repo
There was a problem hiding this comment.
There is no option for that with prettier. You can only set trailing commas for "all", "es5" or "none".
There was a problem hiding this comment.
Please see what is the impact of es5. The objective is to remove trailing commas in function calls, since it's really confusing and sloppy. If es5 does not do what we want we can use none
There was a problem hiding this comment.
"es5" keeps trailing commas for objects and arrays, but does not add them for function parameters.
I think "none" is the closest to what we want.
There was a problem hiding this comment.
"es5" keeps trailing commas for objects and arrays, but does not add them for function parameters.
This seems pretty much what I want
There was a problem hiding this comment.
Ah than I misunderstood! I'll be fixing it right now
AtibQur
force-pushed
the
prettier-conf
branch
from
3841bd9 to
1a3e4ed
Compare
AtibQur
requested review from
alexp-sssup
and removed request for
alexp-sssup
No description provided.