You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When ESLint encounters an exception, it will show a popup warning box in the IDE which must be manually dismissed by the user. For example, when I typed const on a new line in a JS file, I got an error dialog.
Obviously ESLint itself or its plugins (e.g. typescript-eslint) has a bug here and shouldn't be crashing. But IMHO it's not good to subject VSCode IDE users to dialogs caused by bugs in linters that most users can't fix, especially when most of these problems fix themselves once the user edits their code to fix the problem that caused ESLint to crash.
The same is true for "can't load config" errors as described in #1182 (comment). The VSCode user can't do anything to fix someone else's code, so putting up an error box isn't helpful.
IMHO, a better UX would be to report errors without requiring the user to dismiss them. Then, even if ESLint is temporarily broken (or broken for just one file), the developer could still keep working without interruption.
The VSCode Prettier extension handles this nicely. Instead of a modal, it shows status using an icon in the status bar, e.g. - ignored file - OK - file can't be parsed
If the user wants to see details behind problems, they can click the status bar to get taken to the output pane to see details.
Could vscode-eslint do something similar?
The text was updated successfully, but these errors were encountered:
The problem is that these errors usually get unnoticed and users then file issues that something is not working. If such an error occurs it usually isn't anything the extension can fix.
In this specific case though we could ignore the error since it is not a setup problem. But sometimes hard to tell.
When ESLint encounters an exception, it will show a popup warning box in the IDE which must be manually dismissed by the user. For example, when I typed
const
on a new line in a JS file, I got an error dialog.Obviously ESLint itself or its plugins (e.g. typescript-eslint) has a bug here and shouldn't be crashing. But IMHO it's not good to subject VSCode IDE users to dialogs caused by bugs in linters that most users can't fix, especially when most of these problems fix themselves once the user edits their code to fix the problem that caused ESLint to crash.
The same is true for "can't load config" errors as described in #1182 (comment). The VSCode user can't do anything to fix someone else's code, so putting up an error box isn't helpful.
IMHO, a better UX would be to report errors without requiring the user to dismiss them. Then, even if ESLint is temporarily broken (or broken for just one file), the developer could still keep working without interruption.
The VSCode Prettier extension handles this nicely. Instead of a modal, it shows status using an icon in the status bar, e.g.
- ignored file
- OK
- file can't be parsed
If the user wants to see details behind problems, they can click the status bar to get taken to the output pane to see details.
Could vscode-eslint do something similar?
The text was updated successfully, but these errors were encountered: