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: Change the generator-star-spacing options example (fixes #8027) #8034
Conversation
@hoatran1403, thanks for your PR! By analyzing the history of the files in this pull request, we identified @btmills, @IanVS and @mysticatea to be potential reviewers. |
LGTM |
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.
Thanks for contributing! Would you mind updating this with the suggestion in this comment, as well as signing the CLA?
docs/rules/generator-star-spacing.md
Outdated
@@ -59,7 +59,7 @@ The rule takes one option, an object, which has two keys `before` and `after` ha | |||
The default is `{"before": true, "after": false}`. | |||
|
|||
```json | |||
"generator-star-spacing": ["error", {"before": false, "after": true}] | |||
"generator-star-spacing": ["error", {"before": true, "after": false}] |
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.
It looks like this was intended to be a general example of how to configure the rule rather than showing another example of the default configuration. Perhaps something along the lines of:
An example configuration:
"generator-star-spacing": ["error", {"before": true, "after": false}]
We'll probably want a comment before the shorthand configuration on line 72 as well.
6f5a084
to
63161cf
Compare
LGTM |
@kaicataldo Thanks for your review, I just fixed it. |
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 for contributing to ESLint!
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!
What is the purpose of this pull request? (put an "X" next to item)
[ X ] 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
[ ] Other, please explain:
#8027
What changes did you make? (Give an overview)
In file generator-star-spacing.md, Made changes in an example of Options from
"generator-star-spacing": ["error", {"before": false, "after": true}]
to
"generator-star-spacing": ["error", {"before": true, "after": false}]
Is there anything you'd like reviewers to focus on?
This is my first contribution to eslint, i'm not sure whether it matched with all the conventions or not( even though I've followed your pull request guide). Just let me know if there is any conflicts.
Thank you