fix(types): use default syntax for typedef

[robwormald]

this will enable projects consuming symbol-observable from TS (eg, RxJS) to use

import $$observable from 'symbol-observable'

rather than

import * as $$observable from 'symbol-observable'

This might? require downstream changes on RxJS but its more correct methinks.

[kwonoj]

👍 I think that one's my bad.. :/

[SamVerschueren]

I don't think changing the d.ts file is enough. TypeScript expects the default property on the exported object. So I believe this line should be changed to something like this:

var ponyfill = require('./ponyfill')(root);
module.exports = ponyfill;
module.exports.default = ponyfill;

More info: microsoft/TypeScript#2242 (comment)

[benlesh]

@robwormald did you test this? Do we have tests for this? How do we test this? haha. It LGTM... I just want to be sure.

[jamestalmage]

You test it by actually using typescript and the typescript compiler and verifying you get no type warnings / errors.

[benlesh]

You test it by actually using typescript and the typescript compiler and verifying you get no type warnings / errors.

Yes... I know, I'm just travelling and I was hoping to prompt someone else into adding it. :) haha

Too subtle, I guess.

[robwormald]

Maybe don't merge this yet, the behavior is kinda weird. I'll work through it this weekend.

[benlesh]

I just tested this, and it doesn't work properly in TypeScript.

[benlesh]

the import is undefined

[benlesh]

Closing in favor of #12, which adds a test (that is currently failing)