Skip to content

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