Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ui/next: update dependencies #6962

Merged
merged 1 commit into from
May 31, 2016
Merged

ui/next: update dependencies #6962

merged 1 commit into from
May 31, 2016

Conversation

tamird
Copy link
Contributor

@tamird tamird commented May 30, 2016

Notably includes typings 1.0.0.

NB: There are external definitions available for es6-promise, but
isomorphic-fetch's definitions depend on es6-promise's ambient
definitions.

Also note that redux ships with external definitions, but none of the
react packages do, and the react packages' global definitions have not
been updated for use with the new external redux definitions.

Finally, a manual override was necessary to make reselect's external
typings load via plugin-typescript. This appears to be a bug, since
this exact information is present in reselect's package.json.
Discussion is ongoing in frankwallis/plugin-typescript#102.


This change is Reviewable

Notably includes typings 1.0.0.

NB: There are external definitions available for es6-promise, but
isomorphic-fetch's definitions depend on es6-promise's ambient
definitions.

Also note that redux ships with external definitions, but none of the
react packages do, and the react packages' global definitions have not
been updated for use with the new external redux definitions.

Finally, a manual override was necessary to make reselect's external
typings load via plugin-typescript. This appears to be a bug, since
this exact information is present in reselect's package.json.
Discussion is ongoing in frankwallis/plugin-typescript#102.
@maxlang
Copy link
Contributor

maxlang commented May 31, 2016

LGTM, but you'll definitely want to run this by @mrtracy

@mrtracy
Copy link
Contributor

mrtracy commented May 31, 2016

LGTM

@tamird tamird merged commit 5baf8e9 into cockroachdb:master May 31, 2016
@tamird tamird deleted the ui-next-update branch May 31, 2016 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants