Skip to content

Conversation

@tesseralis
Copy link
Member

This PR resolves #33
Cherry picked from reactjs/react.dev@f149987

* add warning for invalid hook call

* Fix versions

* Split code examples

* unnecessary comma

* tweaks

* Update content/warnings/invalid-hook-call-warning.md

Co-Authored-By: gaearon <dan.abramov@gmail.com>

* nit
@tesseralis
Copy link
Member Author

This issue was autogenerated by the bot. @smikitky, do you want to merge as-is and translate it later or would you like someone to translate it now?

@smikitky
Copy link
Member

smikitky commented Feb 2, 2019

@tesseralis I'd rather merge it as-is and assign someone to translate it later. CI is complaining about this line (bracket mismatch; false positive), but we can ignore it

@tesseralis
Copy link
Member Author

@smikitky could you handle that? I'm not familiar with the textlint rules.

@smikitky
Copy link
Member

smikitky commented Feb 2, 2019

@tesseralis Yes, as a repo admin, I can merge anyway. Can I merge this right away?

@tesseralis
Copy link
Member Author

@smikitky Will merging this cause all other PRs to fail from the same reason?

@smikitky
Copy link
Member

smikitky commented Feb 2, 2019

@tesseralis Ah, you're right, then I'll try to find a way to fix it before merge

@smikitky smikitky self-assigned this Feb 2, 2019
@tesseralis tesseralis merged commit 70c0fb3 into master Feb 2, 2019
@smikitky smikitky deleted the f149987d branch February 2, 2019 06:25
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.

[Doc]: Warning page for invalid Hook calls

3 participants