Skip to content

[pull] master from facebook:master#373

Merged
pull[bot] merged 1 commit intochojar:masterfrom
facebook:master
May 29, 2019
Merged

[pull] master from facebook:master#373
pull[bot] merged 1 commit intochojar:masterfrom
facebook:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented May 29, 2019

See Commits and Changes for more details.


Created by pull[bot]

* Generate signatures for Hooks

This currently only works one level deep. For custom Hooks, we'll need to add some way to compose signatures.

* Be more resilient to plugin conflicts

This prevents a class of problems where other plugins cause our visitor to re-run.

It's a standard Babel practice, e.g.:

https://github.com/babel/babel/blob/8c7d4b55c99ff34cb9d493d452472e59b5ed1e70/packages/babel-plugin-transform-react-constant-elements/src/index.js#L85-L86

* Remove unnecessary stuff from debugging

* Include Foo.useHookName() calls into the signature

* Add an integration test for adding/removing an effect

* Add integration test for changing custom Hook order

* Include custom Hooks into the signatures

* Fix inferred names for function expressions

* Support export default hoc(Foo) when Foo is defined separately

* Add more built-in Hooks
@pull pull bot added the ⤵️ pull label May 29, 2019
@pull pull bot merged commit 556cc6f into chojar:master May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant