Skip to content
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

Revert "Fix starting pseudoterminal (#10780)" #12098

Merged
merged 1 commit into from
Jan 23, 2023

Conversation

thegecko
Copy link
Member

Signed-off-by: robmor01 [email protected]

This reverts commit 363e521.

This reverts the previous fix for opening pseudoterminals as they now work as expected.

Supersedes: #12005
Fixes: #11858

Tested in browser and electron examples. Test steps from #12005 can be used (thanks @zvzuola)

cc @arekzaluski @zvzuola

What it does

How to test

Review checklist

Reminder for reviewers

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes work well for me 👍

  • Running the test extension correctly brings up the pseudo terminal

@thegecko thegecko merged commit 1c77006 into eclipse-theia:master Jan 23, 2023
@thegecko thegecko deleted the revert-10780 branch January 23, 2023 15:00
@paul-marechal paul-marechal added this to the 1.34.0 milestone Jan 24, 2023
@thegecko thegecko mentioned this pull request Feb 3, 2023
1 task
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.

Open() method from vscode.Pseudoterminal not called inside Task CustomExecution
3 participants