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.

Sign up for GitHub

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

Jump to bottom

starting "test" Theia instance only works once in a given terminal? #2

Closed
marcdumais-work opened this issue Apr 26, 2018 · 2 comments
Closed

starting "test" Theia instance only works once in a given terminal? #2

marcdumais-work opened this issue Apr 26, 2018 · 2 comments
Labels
type: bug Something isn't working

Comments

@marcdumais-work
Copy link

Reviewing / trying a PR, I used the service. When first started, a terminal was already there, building the PR code. When done, I started a test Theia (yarn run start...) and got the pop-up to open the test instance in a new tab.

Later I restarted the backend, by killing the current one and re-running the command. That time I did not get the pop-up. It however worked when I tried in a fresh terminal.
@svenefftinge svenefftinge added the type: bug Something isn't working label Apr 30, 2018
@marcdumais-work
Copy link
Author

BTW, I retried this with a different PR, and have not seen the issue.

@svenefftinge
Copy link
Member

Yes, it is fixed. It no longer is based on parsing the terminal output.

@jankeromnes jankeromnes mentioned this issue Nov 15, 2018
Closed
@MGlolenstine MGlolenstine mentioned this issue Mar 24, 2020
Closed
@jgallucci32 jgallucci32 mentioned this issue Jan 18, 2021
Closed
fullmetalrooster pushed a commit that referenced this issue Feb 16, 2021
# This is a combination of 8 commits.
86e7486 
# This is the 1st commit message:

dev

# This is the commit message #2:

dev

# This is the commit message #3:

dev

# This is the commit message #4:

dev

# This is the commit message #5:

dev

# This is the commit message #6:

dev

# This is the commit message #7:

dev

# This is the commit message #8:

dev
@jakobhero jakobhero mentioned this issue Aug 12, 2021
Closed
@fntlnz fntlnz mentioned this issue Oct 24, 2021
Closed
@ibrah3m ibrah3m mentioned this issue Apr 19, 2022
Closed
@loujaybee loujaybee mentioned this issue Jun 2, 2022
Closed
easyCZ added a commit that referenced this issue Sep 12, 2022
@easyCZ
# This is a combination of 2 commits.
bf636f5 
# This is the 1st commit message:

[usage] Refactor controller package into scheduler

# This is the commit message #2:

[usage] Refactor controller package into scheduler
roboquat pushed a commit that referenced this issue Sep 13, 2022
@easyCZ @roboquat
# This is a combination of 2 commits.
24c50d2 
# This is the 1st commit message:

[usage] Refactor controller package into scheduler

# This is the commit message #2:

[usage] Refactor controller package into scheduler
iQQBot pushed a commit that referenced this issue Sep 14, 2022
@easyCZ @iQQBot
# This is a combination of 2 commits.
c99cf66 
# This is the 1st commit message:

[usage] Refactor controller package into scheduler

# This is the commit message #2:

[usage] Refactor controller package into scheduler
@jldec jldec mentioned this issue Sep 26, 2022
Closed
@csweichel csweichel mentioned this issue Oct 13, 2022
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@svenefftinge @marcdumais-work