Skip to content
This repository has been archived by the owner on May 17, 2019. It is now read-only.

Add react hook rules and peerDep #136

Merged
merged 3 commits into from
Apr 3, 2019
Merged

Conversation

micburks
Copy link
Contributor

@micburks micburks commented Apr 2, 2019

This is technically breaking since it requires a new peerDep. Unless there is a better way to do this, like installing directly as a dependency.

@ganemone
Copy link
Contributor

ganemone commented Apr 2, 2019

I think it is fine to have a breaking change, you just need to create a file: docs/migrations/00136.md with migration instructions which should just be installation instructions of the new peerDep

@micburks
Copy link
Contributor Author

micburks commented Apr 3, 2019

!merge

@old-fusion-bot old-fusion-bot bot merged commit 7696a31 into master Apr 3, 2019
@old-fusion-bot
Copy link

old-fusion-bot bot commented Apr 3, 2019

Triggered Fusion.js build verification: https://buildkite.com/uberopensource/fusion-release-verification/builds/1815

@old-fusion-bot old-fusion-bot bot deleted the enforce-react-hooks branch April 3, 2019 18:00
This was referenced Apr 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants