7

Closed

Upgrade TypeScript to 0.9.0

description

Upgrade TypeScript to 0.9.0
Closed Dec 3, 2013 at 4:39 AM by mmanela

comments

moly wrote Jun 21, 2013 at 6:35 AM

Upgrade TypeScript to 0.9.0

Halzy wrote Jun 27, 2013 at 5:22 PM

To help jump-start the update:
https://chutzpah.codeplex.com/SourceControl/network/forks/Halzy/TypeScript090/changeset/b647ff5e2fa3

As the commit says, my confidence is low in the correctness.

mmanela wrote Jun 28, 2013 at 1:56 AM

This is great, overall it looks pretty good.
Are you running into issues or is it passing all the integration tests?

Halzy wrote Jun 28, 2013 at 3:46 PM

I think I may be running into a TS 0.9.0 bug, but the bug does not manifest itself with the standalone compiler so it must be how I am invoking it. I am taking another stab at it.

Halzy wrote Jun 29, 2013 at 1:53 AM

TS was updated to 0.9.0.1 today and the result is better. I also re-did the implementation and had to add importing of the *.d.ts files. There are 2 failing tests now, but I think that they just don't work any more due to the compiler being more strict.

https://chutzpah.codeplex.com/SourceControl/network/forks/Halzy/TypeScript090/changeset/4ddf27bff0b8

mmanela wrote Jun 29, 2013 at 3:53 PM

I will test this out and pull it in soon.

Why did you need to add the .d.ts support? Is that working?

Halzy wrote Jun 29, 2013 at 8:30 PM

It didn't recognize the built in types and threw errors for unrecognized functions such as test(). Adding .d.ts support seemed to fix many of those errors.

SalMehdi wrote Jul 1, 2013 at 12:36 PM

hi there, I am seeing an issue when running chutzpah tests that include jQuery.d.ts. Possibly a generics issue. e.g.
Chutzpah console test runner (64-bit .NET 4.0.30319.17929)
Copyright (C) 2013 Matthew Manela (http://matthewmanela.com).

chutzpah.dll: Version 2.4.1.266


Chutzpah Error: E:\includes\jquery\ts\jQuery.d.ts (296,10): Expected ';'

The offending line is.
each<T>(collection: T[], callback: (indexInArray: number, valueOfElement: T) => any): any;

Halzy wrote Jul 1, 2013 at 6:29 PM

Can you check that it compiles correctly in TS 0.9.0.1?

mmanela wrote Jul 11, 2013 at 3:50 AM

fixed in 9da3f7e6c1ec