-
Notifications
You must be signed in to change notification settings - Fork 536
Fix/eslint standarjs conflicts on generator functions #1523
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
Fix/eslint standarjs conflicts on generator functions #1523
Conversation
- fixed generator space linting rule
Also run |
@schnogz thanks! Addressed comments and pushed them. |
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 see we have a warning when running lint tasks: Warning: React version not specified in eslint-plugin-react settings. See https://github.com/yannickcr/eslint-plugin-react#configuration .
Can you add a settings object to .eslintrc
to fix this warning. Add the following snippet and the warning will go away.
"settings": {
"react": {
"version": "latest"
}
},
@schnogz , Could you please share which command generates that warning? Is it on circleci or local? It seems fine on my local. I have tried |
@schnogz Added. Thanks! |
Description
Standard JS requires spacing before and after

*
of a generator function.We could fix this error using auto fix with ESLint's editor plugin (I'm on WebStorm). However, the space will be removed during pre-commit phase (
lint-staged
) and then editor will show lint error again. It becomes cyclic.Change Type
Please enter one or more of the following:
Testing Steps
*
for generator functionsgenerator-star-spacing
. The spacing added in step 1 should remain in code.Code Checklist
yarn start
)README.md
and other documentation is updated as neededFix #1522