-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Error reporting should be smoother #17101
Comments
The markdown supports command links, could that be used to show error details here? |
If you hover on that item you should see the full error details. But I think if we can get a output channel (just like for runs) to report discovery issues that would be great. Since there is already a "Show Output", I think having a way to add discovery errors to that should help with reporting these. /cc @connor4312 Previously we had a custom logging channel that captured both the discovery and run output. Now those two are separated, because there is a way to capture run output for each run. |
It seems like the error is pretty large and textual, maybe it should be written and opened in the editor from a temp file (or custom TextDocumentContentProvider)? |
If users close it accidentally, that error will be lost right? I feel like it needs to be in the logs. Just need to isolate it from the rest of the python logs. Otherwise it might be to difficult to find the issue. /cc @luabud |
we have completed item 2, there is a potential Connor and I might discuss better error reporting but I think for now im going to close this issue since it is stale (pre python testing rewrite), and will move to the updated conversation happening here: #22154 |
Testing #17040
I see the following problems:
The text was updated successfully, but these errors were encountered: