Skip to content

Conversation

@DanielRosenwasser
Copy link
Member

No description provided.

@DanielRosenwasser
Copy link
Member Author

The build failed because of a timeout

 1) fourslash tests tests/cases/fourslash/getEmitOutputDeclarationMultiFiles.ts fourslash test getEmitOutputDeclarationMultiFiles.ts runs correctly: Error: timeout of 20000ms exceeded. Ensure the done() callback is being called in this test. 

I'll trigger a re-test, but this is likely a problem. @mhegazy

@mhegazy
Copy link
Contributor

mhegazy commented Jan 8, 2016

do they locally pass?

@DanielRosenwasser
Copy link
Member Author

Yes, they all pass, it just took longer than 20 seconds for tests/cases/fourslash/getEmitOutputDeclarationMultiFiles.ts to run on the latest version of Node.

DanielRosenwasser added a commit that referenced this pull request Jan 8, 2016
@DanielRosenwasser DanielRosenwasser merged commit 9b151b3 into master Jan 8, 2016
@DanielRosenwasser DanielRosenwasser deleted the lkg-2016-01-08 branch January 8, 2016 18:34
@mhegazy
Copy link
Contributor

mhegazy commented Jan 8, 2016

The last test that @vladima ran Tuesday showed that we have better perf on node v5 than v4. So something does not add up here. Can you get more info?

@DanielRosenwasser
Copy link
Member Author

I think it might've been an anomaly on Travis. The push build was consistent in timings. https://travis-ci.org/Microsoft/TypeScript/builds/101112874

@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

4 participants