Consider renaming `arrowFunction.useParentheses: "always"` - Githubissues

dprint / dprint-plugin-typescript

TypeScript and JavaScript code formatting plugin for dprint.

https://dprint.dev/plugins/typescript

MIT License

248 stars 50 forks source link

Consider renaming `arrowFunction.useParentheses: "always"` #604

Closed mrazauskas closed 2 months ago

mrazauskas commented 5 months ago

Currently the arrowFunction.useParentheses option is the only which takes "force" value. Renaming "force" to "always" would make the API more consistent.

Consider the following:

semiColons

"always" - Always uses semi-colons where applicable.

useBraces

"always" - Forces the use of braces. Will add them if they aren't used.
"preferNone" - Forces no braces when the header is one line and body is one line. Otherwise forces braces.

trailingCommas

"always" - Trailing commas should always be used.
"never" - Trailing commas should not be used.

jsx.multiLineParens

"always" - Always wrap JSX with parentheses if it spans multiple lines.
"never" - Never wrap JSX with parentheses.

And so on and on (;

mrazauskas commented 4 months ago

@dsherret Oh.. Just now I noticed that you added labels. To be sure: does that mean you would merge a PR with this change? I could try putting it together.