Skip to content

Warning page for invalid Hook calls #42

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

Merged
merged 2 commits into from
Feb 2, 2019
Merged

Warning page for invalid Hook calls #42

merged 2 commits into from
Feb 2, 2019

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