Skip to content

Update package.json #153

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

Merged
merged 2 commits into from
Nov 23, 2016
Merged

Update package.json #153

merged 2 commits into from
Nov 23, 2016

Conversation

leftees
Copy link
Contributor

@leftees leftees commented Nov 23, 2016

Add support for eslint-plugin-react-intl plugin to eslint-2 channel.

Add support for eslint-plugin-react-intl plugin to eslint-2 channel.
@@ -27,6 +27,7 @@
"eslint-plugin-jsx-a11y": "1.2.2",
"eslint-plugin-promise": "1.3.1",
"eslint-plugin-react": "5.2.2",
"eslint-plugin-react-intl": "1.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

this indentation looks a little funny. An extra 2 spaces, maybe? Do you mind updating

@maxjacobson
Copy link
Contributor

maxjacobson commented Nov 23, 2016

The missing steps are to run:

npm install && npm shrinkwrap

which will change the npm-shrinkwrap.json file. You'll need to clone and install npm to do that. Let me know if you'd like to go on that adventure or not :)

Edit: turned out that wasn't exactly right, the better thing was to do:

npm install && npm install eslint-plugin-react-intl --save

@maxjacobson maxjacobson mentioned this pull request Nov 23, 2016
@maxjacobson
Copy link
Contributor

LGTM

@maxjacobson maxjacobson merged commit 6c4bfa4 into channel/eslint-2 Nov 23, 2016
@maxjacobson maxjacobson deleted the leftees-eslint-2 branch November 23, 2016 21:39
@maxjacobson
Copy link
Contributor

Rebuilding the merge commit's circle build to trigger the deploy process. Didn't happen last week because of a since-fixed bug in shipbot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants