Added formatToParts to es5.d.ts

[donaldpipowitch]

Added Intl.DateTimeFormat.prototype.formatToParts() to es5.d.ts. See here.

Not sure if tests are mandatory for interface changes?

[msftclas]

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

[mhegazy]

From the internationalization spec: https://tc39.github.io/ecma402/

The ECMAScript 2017 Internationalization API Specification (ECMA-402 4th Edition), provides key languagesensitive functionality as a complement to the ECMAScript 2017 Language Specification (ECMA-262 8th Edition or successor). Its functionality has been selected from that of well-established internationalization APIs such as those of the Internationalization Components for Unicode (ICU) library, of the .NET framework, or of the Java platform.

Based on this i would say these should be added to es2017.d.ts, or a new file es2017.intl.d.ts that is included in es2017.d.ts.

[mhegazy]

also cc @bterlson for his expert opinion.

[mhegazy]

Spoke to @bterlson offline, and he confirmed that the intl is coupled with the ecmascript spec. so i would put these in es2017 instead.

[donaldpipowitch]

Based on this i would say these should be added to es2017.d.ts, or a new file es2017.intl.d.ts that is included in es2017.d.ts.

So I was a little bit unsure about this. es2017.d.ts contained nothing, but imports so I created a new file es2017.intl.d.ts. However it seems that every other es2017.*.d.ts files is referenced in other places, too.

Is this correct?

[mhegazy]

You also need to add it in JakeFile.js

[donaldpipowitch]

Now based on current master. Phew... touched quite a few places.

Run $ npm run build. Seems to work fine.

[mhegazy]

thanks. for future references you do not need to touch files in lib\* these are generated from the build jake LKG.

[donaldpipowitch]

Awesome, thanks.