7

Closed

Upgrade TypeScript to 0.9.0

description

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

comments

moly wrote Jun 21, 2013 at 7:35 AM

Upgrade TypeScript to 0.9.0

Halzy wrote Jun 27, 2013 at 6: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 2: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 4: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 2: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 4: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 9: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 1: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 7:29 PM

Can you check that it compiles correctly in TS 0.9.0.1?

mmanela wrote Jul 11, 2013 at 4:50 AM

fixed in 9da3f7e6c1ec