Skip to content

Fix tsc watch continually restarting during vscode build #3015

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 3 commits into from
Apr 2, 2021

Conversation

code-asher
Copy link
Member

See the commits for more information.

Note that I only moved ipc.d.ts and added a comment to the top but it doesn't show as a rename because the indentation changed from tabs to spaces and git thinks it's a different file.

Now that the plugin API has its own separate types this is no longer
necessary.
It seems reaching into lib/vscode for the types caused tsc to establish
watches that caused it to restart over and over while vscode was
building.

The strategy used here is to symlink it instead which is the same thing
we do for the proxy agent.
@code-asher code-asher requested a review from a team as a code owner April 1, 2021 16:01
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!! This will make local development much more enjoyable.

Non-blocking: wish we had an e2e test to make sure this doesn't cause any regressions (like the ts-node stuff): #2963

@jsjoeio jsjoeio added this to the v3.9.3 milestone Apr 1, 2021
We already lint it on our side.
@code-asher code-asher merged commit 247f11c into coder:main Apr 2, 2021
@code-asher code-asher deleted the fix-tsc-watch-restart branch April 2, 2021 17:10
@jsjoeio jsjoeio added the chore Related to maintenance or clean up label May 14, 2021
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