-
-
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] destructuring-assignment
: add option destructureInSignature
#3235
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3235 +/- ##
==========================================
- Coverage 97.69% 97.67% -0.02%
==========================================
Files 121 121
Lines 8554 8575 +21
Branches 3100 3112 +12
==========================================
+ Hits 8357 8376 +19
- Misses 197 199 +2
Continue to review full report at Codecov.
|
5fb7b13
to
d5e0fda
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.
Looks good overall!
@@ -104,3 +104,33 @@ class Foo extends React.PureComponent { | |||
bar = this.props.bar | |||
} | |||
``` | |||
|
|||
### `destructAtParameter` (default: "ignore") |
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.
maybe destructureInSignature
is better? "destruct" implies it's destroying it, and "parameter" is technically correct but often confused with "argument"
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.
My English destructs.
cf4eaa1
to
ef8c784
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.
Seems great!
destructuring-assignment
: add option destructAtParameterdestructuring-assignment
: add option destructureInSignature
ef8c784
to
567bc7d
Compare
closes #3232.