Skip to content

Support standard style ESLint shared config and plugin #95

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 3 commits into from
Jun 2, 2016

Conversation

dblandin
Copy link
Contributor

@dblandin dblandin commented Jun 2, 2016

Standard Style is a commonly adopted style for writing JavaScript. We hope
to support other third-party plugins and shared configurations in the
future without needing to vendor them in but this update removes a blocker for
a bunch of users and I think it's popular enough to warrant pulling into the
engine.

@codeclimate/review 🔎

@gdiggs
Copy link
Contributor

gdiggs commented Jun 2, 2016

Is this enough for us to not infer a config? If there's no eslintrc we're going to make one

@gdiggs
Copy link
Contributor

gdiggs commented Jun 2, 2016

LGTM

@dblandin dblandin force-pushed the devon/add-standard-plugin-config branch from 467bec6 to e0ca7a4 Compare June 2, 2016 21:28
@dblandin dblandin force-pushed the devon/add-standard-plugin-config branch from e0ca7a4 to a15abbe Compare June 2, 2016 21:36
@dblandin dblandin merged commit e68858b into master Jun 2, 2016
@dblandin dblandin deleted the devon/add-standard-plugin-config branch June 2, 2016 21:52
@Artforge
Copy link

How would I go about configuring this for my application - I've searched around, but find myself at a total loss.

@dblandin
Copy link
Contributor Author

Hey @Artforge,

I believe that you add this to your .eslintrc file:

{
  "extends": "standard"
}

Source: https://github.com/feross/eslint-config-standard#usage

You can also add "standard" to your extends array if you already have one defined:

{
  "extends": ["eslint:recommended", "standard"]
}

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.

4 participants