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

TypeScript@next type acquisition approach may be broken #16609

Closed
johnnyreilly opened this issue Jun 19, 2017 · 7 comments
Closed

TypeScript@next type acquisition approach may be broken #16609

johnnyreilly opened this issue Jun 19, 2017 · 7 comments
Assignees
Labels
Bug A bug in TypeScript

Comments

@johnnyreilly
Copy link

Hey folks!

I'm one of the maintainers of ts-loader

We have a nightly build that runs which includes typescript@next in its matrix. I've noticed this has been failing for a week or so now. See the travis failure here. Note that TypeScript 2.3 and earlier all run fine.

TypeScript Version: nightly (2.5.0-dev.20170618)

Code

The problem is a failing jasmine test here:

https://github.com/TypeStrong/ts-loader/tree/master/test/execution-tests/2.0.3_typesResolution

Expected behavior:

This test should pass - it's not really a runtime test at all; it's purpose is to validate that the @types/jasmine types were acquired.

Actual behavior:

Test fails. Types don't appear to be being acquired.

I'm afraid I don't have too much time to look further into this but it appears that there is something potentially problematic with the TypeScript@next type acquisition approach and I wanted to give you a heads up.

@DanielRosenwasser DanielRosenwasser added the Bug A bug in TypeScript label Jun 19, 2017
@DanielRosenwasser DanielRosenwasser added this to the TypeScript 2.5 milestone Jun 19, 2017
@DanielRosenwasser
Copy link
Member

DanielRosenwasser commented Jun 19, 2017

Hey @johnnyreilly, does this repro in the 2.4 RC?

@johnnyreilly
Copy link
Author

Gimme a mo - I'll find out

@johnnyreilly
Copy link
Author

The build is running now - answer should be yours in the output 😄

@DanielRosenwasser
Copy link
Member

So it sounds like the RC doesn't exhibit this behavior, but something we ported over to 2.4 proper might have broken it. Do you know which specific day the build started failing?

@rbuckton
Copy link
Member

@johnnyreilly, is this resolved as of [email protected], per this comment?

@johnnyreilly
Copy link
Author

Yes it is - closing!

@johnnyreilly
Copy link
Author

@DanielRosenwasser looking back through our nightly builds I think this is the one where it first appeared:

https://travis-ci.org/TypeStrong/ts-loader/builds/243198621

@microsoft microsoft locked and limited conversation to collaborators Jun 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A bug in TypeScript
Projects
None yet
Development

No branches or pull requests

4 participants