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

Consider pulling out Prettier config #4

Open
rix1 opened this issue Sep 18, 2019 · 1 comment
Open

Consider pulling out Prettier config #4

rix1 opened this issue Sep 18, 2019 · 1 comment

Comments

@rix1
Copy link
Member

rix1 commented Sep 18, 2019

Starting from Prettier v1.17 (web is on v1.16.4) it now supports shared configs. @mortnod This means we can move out the prettier config and host it as a separate package. Nice for when you want consistent formatting but don't care about ESlint.

Note: I haven't decided if this is YAGNI or not, just read the release notes and saw that it was supported 👍
@mortnod
Copy link
Contributor

mortnod commented Sep 19, 2019 •
edited
Loading

Cool! Love that they've added this.

Don't know if it's really worth the effort at the moment, though. Perhaps make the change when that distinction is needed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rix1 @mortnod