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.

Sign up for GitHub

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

Jump to bottom

Add ESLint #14

Open
vinitshahdeo opened this issue Oct 10, 2019 · 3 comments
Open

Add ESLint #14

vinitshahdeo opened this issue Oct 10, 2019 · 3 comments
Labels
good first issue Good for newcomers hacktoberfest Send PR and earn a free t-shirt help wanted Extra attention is needed up-for-grabs Solve the issue and raise your first PR
Projects
HacktoberFest 2019

Comments

@vinitshahdeo
Copy link
Owner

Linting is the process of checking the source code for Programmatic as well as Stylistic errors.

Adding ESLint will be helpful in identifying some common and uncommon mistakes that are made during coding.

ESLint covers both code quality and coding style issues.

  • Add ESLint to this project.
  • Add scripts in package.json i.e. npm run test-lint should run lint tests.
  • Do not make any changes to other files.

Please refer this.
@vinitshahdeo vinitshahdeo added this to For Beginners in HacktoberFest 2019 Oct 10, 2019
@vinitshahdeo vinitshahdeo added good first issue Good for newcomers hacktoberfest Send PR and earn a free t-shirt help wanted Extra attention is needed up-for-grabs Solve the issue and raise your first PR labels Oct 10, 2019
@aleoof
Copy link

aleoof commented Oct 10, 2019

i will do this

@vinitshahdeo
Copy link
Owner Author

@aleoof Please go ahead.

@ghost ghost mentioned this issue Oct 10, 2019
Closed
@aleoof
Copy link

aleoof commented Oct 10, 2019

resolved

This was referenced Oct 11, 2019
Closed
Open
wisneycardeal pushed a commit to wisneycardeal/Quotter that referenced this issue Oct 28, 2019
vinitshahdeo#14 Add ESLint
894ef76 
Added [ESLint](https://eslint.org/) on this project.
Added scripts in package.json i.e. npm run test-lint should run lint tests.
[ESLint](https://eslint.org/)
@wisneycardeal wisneycardeal mentioned this issue Oct 28, 2019
Open
@agarwalaman263 agarwalaman263 mentioned this issue Oct 31, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers hacktoberfest Send PR and earn a free t-shirt help wanted Extra attention is needed up-for-grabs Solve the issue and raise your first PR
Projects
HacktoberFest 2019
  
For Beginners
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vinitshahdeo @aleoof