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
Docs: Add prefer-destructuring variable reassignment example #9873
Conversation
add example with a reassignment.
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.
Would you mind adding a line break before your changes, so that it's clear that it's a separate example (and that your two lines of code go together)?
If you wouldn't mind adding a line break between the two var
lines as well, that would be nice, but not necessary.
adding a line break before changes
Closing and reopening to restart Travis/AppVeyor builds. |
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.
LGTM, thanks! Sorry for taking so long to get back to this.
add example with a reassignment.
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[X] Other, please explain: add code example.
What changes did you make? (Give an overview)
Is there anything you'd like reviewers to focus on?