Skip to content
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

"prettier.eslintIntegration": true not working on Windows 10 #107

Closed
GollyJer opened this issue May 30, 2017 · 11 comments
Closed

"prettier.eslintIntegration": true not working on Windows 10 #107

GollyJer opened this issue May 30, 2017 · 11 comments
Labels
bug locked Please open a new issue and fill out the template instead of commenting.

Comments

@GollyJer
Copy link

I've run into this problem with the prettier-eslint-vscode extension and tried yours without any luck as well.

I'm running this version of vscode.
screenshot_07
and prettier gets applied but eslint --fix does not.

@CiGit CiGit added the bug label May 30, 2017
@CiGit
Copy link
Member

CiGit commented May 30, 2017

I've just tried a freshly installed insider on Windows 10 and eslintIntegration without any trouble.
Can you share a minimal input code and your settings which exposes the problem you have?

@GollyJer
Copy link
Author

Hi @CiGit. Thanks for the response!

With that info I was able to track the problem down to, strangely, the Git Lens extension.

Can you install it and see if it breaks for you?

@CiGit
Copy link
Member

CiGit commented May 31, 2017

Tested on Archlinux, stable vscode, a lot of extensions (including git lens) without having errors. Will try on Windows 10 later.
Is this issue relevant to #96 ?

@GollyJer
Copy link
Author

GollyJer commented May 31, 2017

I don't think #96 is related to my issue.

Just to isolate further I disabled every extension except for yours and GitLens and added only "prettier.eslintIntegration": true to my settings.json file.

The problem is 100% reproduce-able.
With GitLens, no eslint --fix. Without GitLens it works as expected.

The same interference happens with @RobinMalfait's extension.

@CiGit
Copy link
Member

CiGit commented May 31, 2017

That's really weird...
Windows 10 + vscode stable + git lens without troubles.

@CiGit
Copy link
Member

CiGit commented May 31, 2017

Please update to latest version, It updates prettier-eslint...
It's worth a try 😆

@GollyJer
Copy link
Author

Definitely weird!
New version doesn't fix anything. :-/

The only thing I can think of is that I'm on vscode-insiders.
I'll do without GitLens for now. 😄

Thanks for trying.

@CiGit
Copy link
Member

CiGit commented Jun 1, 2017

Tried with Insider + Git Lens + eslintIntegration.
I've got the same behavior you described. It seems it also breaks eslint extension.

Quick and Dirty fix

By patching ~\.vscode-insiders\extensions\eamodio.gitlens-3.5.1\node_modules\zone.js\dist\zone-node.js:816

 if (ZoneAwareError[stackRewrite] && error.originalStack) {

to

if (ZoneAwareError[stackRewrite] && typeof error.originalStack === 'string') {

Prettier-vscode runs smoothly. Git lens seems to work too.

@GollyJer
Copy link
Author

GollyJer commented Jun 2, 2017

@CiGit
Copy link
Member

CiGit commented Jun 2, 2017

Thanks.
There is nothing we can do on our side. Closing

@CiGit CiGit closed this as completed Jun 2, 2017
@github-actions
Copy link

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.

@github-actions github-actions bot added the locked Please open a new issue and fill out the template instead of commenting. label Apr 13, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug locked Please open a new issue and fill out the template instead of commenting.
Projects
None yet
Development

No branches or pull requests

2 participants