-
Notifications
You must be signed in to change notification settings - Fork 120
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
style: add prettier for whitespace formatting #142
Conversation
@@ -52,5 +53,11 @@ | |||
"transform": { | |||
"^.+\\.(t|j)sx?$": "@swc/jest" | |||
} | |||
}, | |||
"prettier":{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this config seemed to be what resulted in the least changes from existing code
Hmm, I'm a bit scared prettier and standardjs (/ts-standard) will clash on some code. Is there no way to get this working by adding an extra eslint rule to the standardjs ones? Or otherwise, |
I've seen many repos with all 3 (ts checking, eslint, and prettier), there shouldn't be any clashing I don't think? The problem with |
I think it's definitely possible to add extra rules and configure |
Have tried to get multiple plugins to work with |
Thought I'd try this out and see how it looked, since ts-standard wasn't fixing whitespaces. Just a proposal, up to the repo owners if they like this!