Skip to content

Fix watcher not restarting code-server on VS Code compilation #4520

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

Merged
merged 1 commit into from
Nov 16, 2021

Conversation

code-asher
Copy link
Member

The "Starting watch-client" string no longer appears in the latest build
output. We could look for "Finished compilation with" to avoid
restarting when other tasks restart (since they also include the name
i.e. "Finished compilation extensions with 0 errors") but I figure we
might as well restart code-server when any compilation task completes in
case other tasks include changes that need to be reloaded.

The "Starting watch-client" string no longer appears in the latest build
output.  We could look for "Finished compilation with" to avoid
restarting when other tasks restart (since they also include the name
i.e. "Finished compilation extensions with 0 errors") but I figure we
might as well restart code-server when any compilation task completes in
case other tasks include changes that need to be reloaded.
@code-asher code-asher requested a review from a team as a code owner November 16, 2021 21:47
@jsjoeio jsjoeio added the chore Related to maintenance or clean up label Nov 16, 2021
@jsjoeio jsjoeio added this to the 4.0.0 milestone Nov 16, 2021
Copy link
Contributor

@jsjoeio jsjoeio left a comment

Choose a reason for hiding this comment

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

Nice cleanup!

@codecov
Copy link

codecov bot commented Nov 16, 2021

Codecov Report

Merging #4520 (30ce10a) into main (ccb9d94) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4520   +/-   ##
=======================================
  Coverage   66.03%   66.03%           
=======================================
  Files          30       30           
  Lines        1640     1640           
  Branches      330      330           
=======================================
  Hits         1083     1083           
  Misses        474      474           
  Partials       83       83           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ccb9d94...30ce10a. Read the comment docs.

@code-asher code-asher merged commit cd26f84 into coder:main Nov 16, 2021
@code-asher code-asher deleted the fix-watch branch November 16, 2021 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Related to maintenance or clean up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants