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

Don't show "ESLint" in the status bar when a non-applicable file is open #1189

Closed
brettcannon opened this issue Feb 27, 2021 · 6 comments
Closed
Labels
info-needed Issue requires more information from poster

Comments

@brettcannon
Copy link
Member

Based on #1068 I assume that minimizing the activation scope to specific file types isn't desired (but maybe I'm wrong?), but would it be possible to not show "ESLint" in the status bar when you have e.g. a .rs file open?

@dbaeumer
Copy link
Member

dbaeumer commented Mar 1, 2021

The problem here is that we can only decide whether ESLint will validate a file if the user gave permission to run the ESLint library (the status bar item has NO blocked sign). Is this the case for you? If so can you please provide me with a GitHub repository I can clone that demos what you are experiencing.

@dbaeumer dbaeumer added the info-needed Issue requires more information from poster label Mar 1, 2021
@brettcannon
Copy link
Member Author

Unfortunately I can only reliably trigger it on a personal, private repo, but it's causing the "✔✔ ESLint" to show when opening the README.md file (the repo itself only contains JSON and Python files otherwise).

@dbaeumer
Copy link
Member

dbaeumer commented Mar 5, 2021

OK, the two check marks explain it. You have allowed ESLint to run everywehere. To signal this to the user we show ESLint status icon always when running in the mode.

@brettcannon
Copy link
Member Author

So is the expectation that I install it disabled and then manually enable it per workspace?

@dbaeumer
Copy link
Member

dbaeumer commented Mar 8, 2021

Yes and No. The problem is that I need to show an indication when Always Allow was selected to indicate the mode. Instead of having another status item we decided to show the ESLint one instead. However this will change when ESLint will adopt the trusted workspace model VS Code is working on.

@brettcannon
Copy link
Member Author

Ah, OK, now I get it: because I said I trusted ESLint you're being upfront by visually indicating it's always running so people are aware of that fact. And since ESLint is being triggered for me by e.g. README.md it's showing up all the time even for repos with zero JS/TS.

I'm going to go ahead and close this since the trusted workspaces approach totally solves the reason I brought this up. I can disable the extension as necessary for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info-needed Issue requires more information from poster
Projects
None yet
Development

No branches or pull requests

2 participants