Skip to content

Conversation

@mDibyo
Copy link
Contributor

@mDibyo mDibyo commented Nov 5, 2018

Griddle major version

v1

Changes proposed in this pull request

Upgrade recompose to ^0.25.1

Why these changes are made

This is the first version of recompose "officially" compatible with React 16.
(acdlite/recompose#530 (comment))

Griddle react seems to work fine in React 16 without this change.

#823 would achieve the same result, but seems to have failing CI tests.
This is minimal change required to resolve warning "griddle-react > [email protected]" has incorrect peer dependency from npm/yarn.

Are there tests?

No additional tests.

This is the first version of recompose compatible with React 16
(acdlite/recompose#530 (comment))
@mDibyo
Copy link
Contributor Author

mDibyo commented Nov 5, 2018

CI is failing in npm run check-ts step, due to unrelated reasons.
#840 fixes CI.

@mDibyo mDibyo mentioned this pull request Nov 5, 2018
@mDibyo
Copy link
Contributor Author

mDibyo commented Nov 10, 2018

After merging in #840, specs pass

@ibrahima
Copy link
Contributor

Hi! Is there any chance that this could be merged? It looks like specs pass now and it seems like there should not be any significant changes: https://github.com/acdlite/recompose/releases?after=v0.25.1

I think technically Griddle can be used on React 16 already, but you will get warnings during npm/yarn install about incorrect peer dependency versions.

Thanks!

@ryanlanciaux ryanlanciaux merged commit 49306e7 into GriddleGriddle:master Dec 4, 2018
@mDibyo mDibyo deleted the dibyo/upgrade-recompose 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.

3 participants