-
Notifications
You must be signed in to change notification settings - Fork 34
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
Add README instructions to run ESLint locally #50
Conversation
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.
Approved with comment
@jayrevolinskyjr please try to locate one person from #47 who can run the installation command to confirm it works.
If they already ran it locally, then it's fine to proceed.
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.
Works for me!
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 Works!
Works for me! |
If reviewing my pull request, please follow instructions shown in changed files or in the 'run_lint_local' branch in the README.md under "Static Build Analysis Tools" for "ESLint" to confirm that it works on your local system.
Should 'running ESLint locally' have it's own section entirely? Currently, I took the liberty to just place it in the "Static Build Analysis Tools" section of the README.md
Completes the 4th task necessary for Issue #47.