-
Notifications
You must be signed in to change notification settings - Fork 8.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
Show a more descriptive error message when the ConptyConnection fails to start for some reason #7186
Comments
How about just displaying the error message that matches the error code, like so:
Or does Windows Terminal have to recognize the 0x800702e4 code specifically and display something more verbose? |
I think displaying the error message would be perfectly fine. I had to google it. With the error message, this wouldn't have been necessary. |
Could even extend that to process exit codes that look like NTSTATUS error codes:
That would have a greater risk of false matches, though. |
This isn't a bad idea, and I bet @DHowett thought about something like this when he first added the error codes |
Yeah, we can definitely do better with these error messages! |
For those who'd like to try this themselves, I added a similar kind of error message in #10045. Collected relevant error messages:
|
Description of the new feature/enhancement
Show a speaking error message, when it is necessary to run Terminal with admin privileges. See #4272 for the problem
Proposed technical implementation details (optional)
The text was updated successfully, but these errors were encountered: