-
-
Notifications
You must be signed in to change notification settings - Fork 171
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
Eslint config based prettying #48
Comments
Hi there! Here's the problem. These should probably have a default value (learn more about destructuring here, learn about default values specifically here). How would you like to add another test case or two here and implement those fixes? Thanks! |
@kentcdodds sounds like a plan for spare time at the weekend ;) |
That line of code has moved to here do you still wanna do this? |
@kentcdodds sure but unfortunately I didn't have spare time, I will do it this week |
@kentcdodds is there any way of testing if my code is fixing that in action and I probably will need your help with test coverage :) |
Add a test case or so here: prettier-eslint/src/utils.test.js Line 170 in a7cc1de
Thanks! |
@kentcdodds thanks for opportunity and apologize for delay :) |
Hey,
based on:
prettier/prettier-atom#85 (comment)
there is something wrong with reading eslint configuration.
Used configuration
and output from
prettier-eslint-cli
:The text was updated successfully, but these errors were encountered: