-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[New] hook-use-state
: add allowDestructuredState
option
#3449
[New] hook-use-state
: add allowDestructuredState
option
#3449
Conversation
19ec70e
to
fa72d87
Compare
Codecov Report
@@ Coverage Diff @@
## master #3449 +/- ##
=======================================
Coverage 97.62% 97.62%
=======================================
Files 123 123
Lines 8978 8978
Branches 3282 3282
=======================================
Hits 8765 8765
Misses 213 213 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When viewing the diff ignoring whitespace changes, it looks great - nice and simple. Thanks!
fa72d87
to
54f0f84
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to hold off on landing semver-minor things until after I've cut the next patch release, but hopefully that won't be long.
54f0f84
to
a258d1d
Compare
hook-use-state
: add allowDestructuredState
option
a258d1d
to
c3d3e25
Compare
This PR fixes #3443 by adding a new option to the rule.
I want to be sure that I'm on the right path, so I could add fixes later.