Skip to content

Add extra check to tsdk path when getting version#19886

Merged
mjbvz merged 1 commit intomicrosoft:masterfrom
mjbvz:add-extra-check-for-ts-server
Feb 3, 2017
Merged

Add extra check to tsdk path when getting version#19886
mjbvz merged 1 commit intomicrosoft:masterfrom
mjbvz:add-extra-check-for-ts-server

Conversation

@mjbvz
Copy link
Copy Markdown
Collaborator

@mjbvz mjbvz commented Feb 3, 2017

Fixes #19600

Ensures that a valid tsserver.js file exists when we check if a tsdk path is valid

@mjbvz mjbvz requested a review from kieferrm February 3, 2017 23:23
@mention-bot
Copy link
Copy Markdown

@mjbvz, thanks for your PR! By analyzing the history of the files in this pull request, we identified @dbaeumer and @egamma to be potential reviewers.

@mjbvz mjbvz force-pushed the add-extra-check-for-ts-server branch from 0061d35 to 6bfe94e Compare February 3, 2017 23:26
@mjbvz mjbvz force-pushed the add-extra-check-for-ts-server branch from 6bfe94e to f2e0f6b Compare February 3, 2017 23:26
@mjbvz mjbvz changed the title Add extra check tsdk path when getting version Add extra check to tsdk path when getting version Feb 3, 2017
@mjbvz
Copy link
Copy Markdown
Collaborator Author

mjbvz commented Feb 3, 2017

This ensures we get this prompt if the last section of the path is incorrect:

screen shot 2017-02-03 at 3 33 59 pm

@mjbvz mjbvz merged commit de28ea6 into microsoft:master Feb 3, 2017
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle typescript.tsdk path does not exist with wrong sub directory better

3 participants