Hello!
I noticed that some errors have showUser set to false. In spite of that, VSCode shows these errors anyway (although maybe it shouldn't?). That's actually nice, since the errors are often helpful. However, since it introduces this inconsistency, the errors should instead have showUser set to true.
One example are the REPL errors, as can be seen here:
|
export const replError = (message: string) => createSilentError(message, ErrorCodes.ReplError); |
Thoughts?
Sample screenshot
Hello!
I noticed that some errors have
showUserset to false. In spite of that, VSCode shows these errors anyway (although maybe it shouldn't?). That's actually nice, since the errors are often helpful. However, since it introduces this inconsistency, the errors should instead haveshowUserset to true.One example are the REPL errors, as can be seen here:
vscode-js-debug/src/dap/errors.ts
Line 156 in 7470237
Thoughts?
Sample screenshot