Skip to content

Stop using gulp-tsc, start using gulp-typescript #17

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

Closed
wants to merge 1 commit into from

Conversation

ryanmolden
Copy link

See issue #16

@bryanmacfarlane
Copy link
Contributor

I used to use gulp-typescript and had issues every time new and preview drops of typescript came out. Turns out calling tsc (that I install) just works.

@ryanmolden
Copy link
Author

That confuses me. gulp-tsc uses where to find tsc.exe, which means it ends up (on a Windows box with VS installed) calling the TSC that VS installs, which is 1.0. If I upgrade VS your build using gulp-tsc suddenly starts using a new compiler, so gulp-tsc does not isolate you from new/preview versions, quite the opposite.

gulp-typescript on the other hand uses the MS published TS compiler from npm, so if you have a set dependency on gulp-typescript version X then you ALWAYS will have a typescript compiler on the box that matches version X of gulp-typescript and it won't matter if I upgrade or even uninstall VS, your code will continue to build and be locked to a specific TS compiler version.

Is it because you were saying >= for your gulp-tsc dependency? I could see auto TS updates in that situation, but you're no more isolated than if VS Update X decides to push out a new TS compiler, unless you don't install VS on your box, or the TS tools apart from whatever one you happen to know works with the project.

@bryanmacfarlane
Copy link
Contributor

that sounds like an issue with VS. It's simple - tsc just calls the typescript on your box which is what I'm after (allows me to pick up newer ts versions).

@bryanmacfarlane
Copy link
Contributor

we ended up switching to gulp-typescript later. thx

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

Successfully merging this pull request may close these issues.

2 participants