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.

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

Embed TypeScript Runtime #24

Closed
angelozerr opened this issue Aug 11, 2017 · 6 comments
Closed

Embed TypeScript Runtime #24

angelozerr opened this issue Aug 11, 2017 · 6 comments
Labels

Comments

@angelozerr
Copy link
Contributor

If #22 is done, we could provide ambed TypeScript Runtime that user could change to use last TypeScript version (like I have done with typescript.java).

@mickaelistria
Copy link
Contributor

mickaelistria commented Aug 11, 2017 via email

@angelozerr
Copy link
Contributor Author

I believe simply
expecting tsserver to be in PATH and warn if missing would be enough.

Customize the TypeScript sdk/runtime should be very good like VSCode does.

@mickaelistria
Copy link
Contributor

mickaelistria commented Aug 18, 2017 via email

@angelozerr
Copy link
Contributor Author

Embedding TS runtime is clearly something we should avoid as it causes more
issues that it fixes. Most ts developers have installed TS runtime by
themselves.

VSCode works like this. It provides a default typescript sdk.

However, providing a preference to let end-user define another location for
TS runtime could be nice.

With angular project generated with angular-cli, the typescript runtime is installed in the node_modules of your project. It's better to consume it instead of consuming an another ts runtime.

@mickaelistria
Copy link
Contributor

mickaelistria commented Aug 19, 2017 via email

@mickaelistria
Copy link
Contributor

I think we can close this one.
With #22 , typescript is now included in the dependencies and it seems like it's not a requirement for usage any more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants