Skip to content

Add terminal information to chooseTerminal error#447

Merged
Ikuyadeu merged 2 commits intoREditorSupport:masterfrom
andycraig:choose-terminal-log
Nov 16, 2020
Merged

Add terminal information to chooseTerminal error#447
Ikuyadeu merged 2 commits intoREditorSupport:masterfrom
andycraig:choose-terminal-log

Conversation

@andycraig
Copy link
Copy Markdown
Collaborator

@andycraig andycraig commented Nov 14, 2020

What problem did you solve?

This is related to #385. I can't reproduce the error, so this PR adds more information to the error message, which users can pass on to us.

Screenshot

I can't reproduce the error, so I made this screenshot by copy-pasting the message code to the top of chooseTerminal().

EDIT: Updated the image.

terminals3

How can I check this pull request?

I don't have instructions for how to reproduce the error, so I think review will have to be code review only, without testing.

@andycraig andycraig marked this pull request as draft November 14, 2020 09:46
@andycraig andycraig changed the title Add terminal information to chooseTerminal error WIP Add terminal information to chooseTerminal error Nov 14, 2020
@andycraig andycraig changed the title WIP Add terminal information to chooseTerminal error Add terminal information to chooseTerminal error Nov 14, 2020
@andycraig andycraig marked this pull request as ready for review November 14, 2020 10:09
@andycraig
Copy link
Copy Markdown
Collaborator Author

Ready for review.

Comment thread src/rTerminal.ts
@Ikuyadeu
Copy link
Copy Markdown
Member

Good! It can be merged.
I put only one comment, but it is not so important and you can ignore it.

@Ikuyadeu Ikuyadeu merged commit c285e7f into REditorSupport:master Nov 16, 2020
@andycraig
Copy link
Copy Markdown
Collaborator Author

@Ikuyadeu Thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants