Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

馃搸 Add arrowParentheses formatter option #4666

Closed
SuperchupuDev opened this issue Jul 7, 2023 · 0 comments 路 Fixed by #4667
Closed

馃搸 Add arrowParentheses formatter option #4666

SuperchupuDev opened this issue Jul 7, 2023 · 0 comments 路 Fixed by #4667
Assignees
@ematipico @denbezrukov @SuperchupuDev
Labels
A-Formatter Area: formatter L-JavaScript Langauge: JavaScript task A task, an action that needs to be performed
Milestone
v13.0.0

Comments

@SuperchupuDev
Copy link
Contributor

Description

Relevant: #3910

Prettier has the option to remove unnecesary parentheses from arrow functions, and it's a widely used option, in fact the linked gh discussion for this repo is currently the third most upvoted discussion in here.

Github Search shows that out of 46k .prettierrc files, 10.6k of them have arrowParens set to avoid (last time I checked at least), which is 23% of them using this option. It's used a lot and that's why it's important to have it in Rome

image
image
@SuperchupuDev SuperchupuDev added the task A task, an action that needs to be performed label Jul 7, 2023
@SuperchupuDev SuperchupuDev mentioned this issue Jul 7, 2023
Merged
@ematipico ematipico added A-Formatter Area: formatter L-JavaScript Langauge: JavaScript labels Jul 10, 2023
@ematipico ematipico added this to the v13.0.0 milestone Jul 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ematipico ematipico

@denbezrukov denbezrukov

@SuperchupuDev SuperchupuDev

Labels
A-Formatter Area: formatter L-JavaScript Langauge: JavaScript task A task, an action that needs to be performed
Projects
None yet
Milestone
v13.0.0
Development

Successfully merging a pull request may close this issue.

feat(rome_js_formatter, rome_cli): add arrowParentheses option
3 participants
@ematipico @denbezrukov @SuperchupuDev