-
Notifications
You must be signed in to change notification settings - Fork 94
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 upgrading prettier/eslint-config-prettier/eslint-plugin-prettier #451
Comments
Hi @davidkaneda , Can you post your .codeclimate.yml here? Happy to take a closer look. 👌 Thanks! |
Thanks, here you go:
|
We actually have a quite similar issue. For us Running this locally, gives no errors: Config:
@davidkaneda could you resolve the issue at your end? |
Never mind. Issue seems fixed by adding |
You need to install an exact Prettier version.
And I suspect further complications related to |
When you set |
My apologies for not including steps to reproduce and such, but my repo is private and I'm just getting started with codeclimate. Essentially, I'm getting a handful of linter errors in codeclimate that I'm not locally/in Circle, and I believe it's due to the mismatch in dependencies.
My local dependencies:
Here's a sample of the errors:
I have a sort of reverse reproduction in that if I go through and pin those 3 prettier-related deps to the versions in this repo, I get the lint errors (actually, the ones in my output plus a few more). Note that
prettier
itself actually resolves to the same version, as it hasn't had a major update like the two others.Not sure if you have a policy on these sorts of upgrades (as I'm sure fixing for me will break others), but figured I'd give it a shot. Thanks!
The text was updated successfully, but these errors were encountered: