-
-
Notifications
You must be signed in to change notification settings - Fork 448
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
Consider adding back prettier-eslint local resolution #108
Comments
I would agree on updating it. Requiring a local version is a potential trouble where we have no control. I prefer avoid that to be honest :-) Having it work out of the box is pleasant IMO. |
Just published 0.17.2 with prettier-eslint@6.2.3 |
Hmm ok, I'm not going to pursue but still think it would be nice to ensure that e.g. the output from the extension is always the same as from the cli. Thanks for the update. |
I've let that one opened for discussion. |
stalled, closing |
+1 to the idea of using the local version of I have upgraded to v9 (which has bug fixes) whilst this extension is stuck on v8. This means that formatting via the CLI vs via VS Code has different results, which is very confusing until you understand why this is. Is there any reason not to use the local |
This seems to have caused the issue in #861 |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
I.e. 4cc8d01
Or keep
prettier-eslint
always updated.It's another extension (after
prettier-eslint-code
) I'm having trouble with using because it stays with an outdated dependency. Right nowprettier-eslint
is at6.2.3
(with some bugs fixed) but can't be used because this extension locked it at6.1.2
.Here you decided that the output from
prettier-eslint
will not change between versions but what else could change if not the output (if its only function is formatting)?I would even say it makes more sense to not bundle
prettier-eslint
at all and just rely on the user to provide one either globally or in the workspace (that's what eslint extension does). It gives control to the user and doesn't lead to unexpected results.The text was updated successfully, but these errors were encountered: