Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[all] Change prettier config arrowParens: avoid #4819

Merged
merged 4 commits into from
Jul 8, 2020
Merged

Conversation

styfle
Copy link
Member

@styfle styfle commented Jul 8, 2020

In PR #4760, Prettier was upgraded to 2.0 which changed the default configuration for arrowParens from avoid to always. This PR changes it back to avoid so we don't have PRs changing unnecessary lines during in the commit hook.

See the Prettier 2.0 Announcement for more.

@TooTallNate
Copy link
Member

Can you run prettier on the CLI test runner files?

AndyBitz
AndyBitz previously approved these changes Jul 8, 2020
@styfle styfle changed the title [all] Change prettier config arrowParens: 'avoid' [all] Change prettier config arrowParens: avoid Jul 8, 2020
@styfle
Copy link
Member Author

styfle commented Jul 8, 2020

@TooTallNate Done 👍

@styfle styfle requested a review from AndyBitz July 8, 2020 22:29
@styfle styfle merged commit 47e3111 into master Jul 8, 2020
@styfle styfle deleted the prettier-avoid-parens branch July 8, 2020 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants