Skip to content
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

Merged
merged 3 commits into from
Mar 9, 2023

Conversation

jayrevolinskyjr
Copy link
Contributor

@jayrevolinskyjr jayrevolinskyjr commented Mar 8, 2023

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.

@jayrevolinskyjr jayrevolinskyjr added documentation Improvements or additions to documentation DevOps Concerns developer operations labels Mar 8, 2023
@Jooms Jooms mentioned this pull request Mar 8, 2023
4 tasks
@jayrevolinskyjr jayrevolinskyjr changed the title #47 - add README instructions to run ESLint locally Add README instructions to run ESLint locally Mar 8, 2023
Jooms
Jooms previously approved these changes Mar 8, 2023
Copy link
Contributor

@Jooms Jooms left a 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.

Copy link
Contributor

@ysiddhapura ysiddhapura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me!

Copy link
Contributor

@parthpandey1 parthpandey1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It Works!

@jayrevolinskyjr jayrevolinskyjr merged commit cc935f1 into main Mar 9, 2023
@jayrevolinskyjr jayrevolinskyjr deleted the run_lint_local branch March 9, 2023 01:41
@Anoushka444
Copy link
Contributor

Works for me!

briswells pushed a commit that referenced this pull request Mar 23, 2023
Completes the 4th task necessary for Issue #47. 

- [ ] Clean up **@typescript-eslint/no-var-requires**
- [ ] #48
- [ ] Clean up **no-undef**
- [x] Add "how to run the linter locally" to the README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DevOps Concerns developer operations documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants