Skip to content

eslint command added to package.json#2625

Closed
Morkowski wants to merge 1 commit into
react:masterfrom
Morkowski:eslint-command
Closed

eslint command added to package.json#2625
Morkowski wants to merge 1 commit into
react:masterfrom
Morkowski:eslint-command

Conversation

@Morkowski

Copy link
Copy Markdown

Ability to run Eslint from command contained in package.json

Connected with discussion: #1217

build: 'react-scripts build',
test: 'react-scripts test --env=jsdom',
eject: 'react-scripts eject',
eslint: 'node node_modules/eslint/bin/eslint.js --config node_modules/eslint-config-react-app/index.js',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe just lint?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yep.

@gaearon

gaearon commented Jun 27, 2017

Copy link
Copy Markdown
Contributor

This is not quite what I meant.

I don't want to add a new entry to scripts, as I said in #1217 (comment):

We won't add it to the project scripts by default

But I want the script in bin/react-scripts to work so that people who need it can add react-scripts lint.

@gaearon gaearon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See above comment

@devonjs devonjs mentioned this pull request Jul 6, 2017
@Timer

Timer commented Aug 2, 2017

Copy link
Copy Markdown
Contributor

I believe this has moved to #2729.

@Timer Timer closed this Aug 2, 2017
@lock lock Bot locked and limited conversation to collaborators Jan 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants