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

Upgrade typescript to ^2.8.0 #840

Merged

Conversation

mDibyo
Copy link
Contributor

@mDibyo mDibyo commented Nov 5, 2018

Griddle major version

v1

Changes proposed in this pull request

Upgrade typescript to ^2.8.0

Why these changes are made

This should hopefully fix error in travis build step npm run check-ts

Error:

node_modules/@types/recompose/index.d.ts(23,47): error TS2304:
Cannot find name 'Exclude'.

Failing builds:
https://travis-ci.org/GriddleGriddle/Griddle/builds/441174637
https://travis-ci.org/GriddleGriddle/Griddle/builds/450669719

Reason: The Exclude keyword used in @types/recompose was added in
Typescript 2.8
(https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-8.html#predefined-conditional-types).

Are there tests?

Successful CI is the test.

This should hopefully fix error in travis build step `npm run check-ts`

Error: node_modules/@types/recompose/index.d.ts(23,47): error TS2304:
Cannot find name 'Exclude'.
Example build:
https://travis-ci.org/GriddleGriddle/Griddle/builds/441174637

Reason: The Exclude keyword used in @types/recompose was added in
Typescript 2.8
(https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-8.html#predefined-conditional-types).
@ryanlanciaux ryanlanciaux merged commit cc296fe into GriddleGriddle:master Nov 8, 2018
@mDibyo mDibyo deleted the dibyo/upgrade-typescript branch December 24, 2018 07:35
mDibyo added a commit to mDibyo/Griddle that referenced this pull request Dec 24, 2018
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.

2 participants