Typescript: Fix DefinitelyTyped type definitions before publishing a new version

Created on 23 Feb 2017  路  6Comments  路  Source: microsoft/TypeScript

Problem:
Each TypeScript release breaks popular DefinitelyTyped type definitions, example with TypeScript 2.2 release:

Not a good developer experience

Solution:
Fix the most popular DefinitelyTyped type definitions before publishing on npm

Infrastructure

Most helpful comment

As for the original request, we used to validate against every drop of the compiler but as the size of DT grew it became harder to do that.
I agree We should be running the validation on all DT packages at least once before every release. @andy-ms is updating the testing system to make this doable.

All 6 comments

Given the issue with loads at least was that the change would break older versions of TS. Maybe if there was some sort of @next option for DefinitelyTyped that way at least we call you have updated to that version of the definition when the RC has launched. That would have made things a little easier at least.

When can we expect isomorphic-fetch type fixes to be deployed? Waiting on this currently...

@types/[email protected] is already out and should have the fix.

For lodash, the change https://github.com/DefinitelyTyped/DefinitelyTyped/pull/14662 needs to be updated and then we can merge it.

As for the original request, we used to validate against every drop of the compiler but as the size of DT grew it became harder to do that.
I agree We should be running the validation on all DT packages at least once before every release. @andy-ms is updating the testing system to make this doable.

We do this regularly now, thanks to the @typescript-bot run dt command (not just before publish, but regularly before merge), in addition to nightly testing of DT vs the nightly TS. 馃槃

Was this page helpful?
0 / 5 - 0 ratings

Related issues

siddjain picture siddjain  路  3Comments

wmaurer picture wmaurer  路  3Comments

dlaberge picture dlaberge  路  3Comments

seanzer picture seanzer  路  3Comments

MartynasZilinskas picture MartynasZilinskas  路  3Comments