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 recompose to ^0.25.1 #839

Merged
merged 2 commits into from
Dec 4, 2018

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 > recompose@0.21.2" 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