-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
Setup Prettier #162
Setup Prettier #162
Conversation
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.
Thanks a lot for picking this one up! 👏
Would it okay for you to align it more with Prettier RFC? I think this would be more future proof and cause less noise when Prettier is added to default blueprints.
package.json
Outdated
@@ -19,7 +19,8 @@ | |||
"start": "ember serve", | |||
"test": "ember test", | |||
"test:all": "ember try:each", | |||
"test:node": "mocha node-tests/**/*-test.js" | |||
"test:node": "mocha node-tests/**/*-test.js", | |||
"format": "prettier --write \"./**/*.{md,json,js,yml}\"" |
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.
I would prefer to setup Prettier as ESLint plugin as described in the RFC: https://emberjs.github.io/rfcs/0628-prettier.html#detailed-design My main motivation is that we don't need to change much when this RFC is implemented in Ember CLI.
I'll implement all the changes |
implemented prettier with rfc in mind
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.
Thanks a lot!
Fixes issue #157
-added .pretterrc.json
-added prettier script in package.json
-the script was ran on the repo
Please let me know if I need to make changes