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.

Sign up for GitHub

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

Jump to bottom

Align (a bit more) with prettier 2 defaults #178

Merged
merged 2 commits into from
Apr 9, 2020
Merged

Align (a bit more) with prettier 2 defaults #178

merged 2 commits into from
Apr 9, 2020

Conversation

papandreou
Copy link
Collaborator

@papandreou papandreou commented Apr 6, 2020 •
edited
Loading

In the Unexpected org we decided to give the new defaults (trailingComma and arrowParens) a try while we're at it. I thought we might want to do the same here :)

@papandreou papandreou self-assigned this Apr 6, 2020
@papandreou papandreou requested a review from Munter April 6, 2020 09:59
@coveralls
Copy link

Coverage Status

Coverage remained the same at 77.573% when pulling 5ae520e on tech/alignWithPrettier2Defaults into 70f203e on master.

@Munter
Copy link
Owner

Munter commented Apr 9, 2020

👍

@Munter Munter merged commit 8a763aa into master Apr 9, 2020
@Munter Munter deleted the tech/alignWithPrettier2Defaults branch April 9, 2020 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Munter Munter Awaiting requested review from Munter

Assignees

@papandreou papandreou

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@papandreou @coveralls @Munter