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

setup: setup eslint and husky #40

Merged
merged 1 commit into from Aug 15, 2019
Merged

setup: setup eslint and husky #40

merged 1 commit into from Aug 15, 2019

Conversation

karansapolia
Copy link
Contributor

fixes #3

@karansapolia karansapolia added the Setup Contains data related to setting up the project. label Aug 14, 2019
@karansapolia karansapolia self-assigned this Aug 14, 2019
@karansapolia karansapolia added this to In progress in Lumen via automation Aug 14, 2019
Copy link
Contributor

@nihalanisumit nihalanisumit left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@piyushagwl007 piyushagwl007 left a comment

Choose a reason for hiding this comment

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

LGTM. Just need some explanation for some things 😃

@@ -13,9 +13,6 @@
"test": "react-scripts test",
"eject": "react-scripts eject"
},
"eslintConfig": {
"extends": "react-app"
},
"browserslist": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did you remove it?

package.json Show resolved Hide resolved
.eslintignore Show resolved Hide resolved
Lumen automation moved this from In progress to Reviewer approved Aug 15, 2019
@karansapolia karansapolia merged commit 5566c0c into dev Aug 15, 2019
Lumen automation moved this from Reviewer approved to Done Aug 15, 2019
@karansapolia karansapolia deleted the karan/setup-eslint branch August 26, 2019 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Setup Contains data related to setting up the project.
Projects
Lumen
  
Done
Development

Successfully merging this pull request may close these issues.

Setup eslint , husky
4 participants