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

Allow use of eslint in package.json #2670

Closed
sindresorhus opened this issue Jun 2, 2015 · 5 comments
Closed

Allow use of eslint in package.json #2670

sindresorhus opened this issue Jun 2, 2015 · 5 comments
Labels
archived due to age This issue has been archived; please open a new issue for any further discussion triage An ESLint team member will look at this issue soon

Comments

@sindresorhus
Copy link
Contributor

Currently you can specify "eslintConfig": {} in package.json with ESLint config.

It would look a lot nicer it was named "eslint": {}. ESLint could just try reading both.

I know this is minor, but anything to make package.json less verbose and nicer looking.

@ilyavolodin ilyavolodin added the triage An ESLint team member will look at this issue soon label Jun 2, 2015
@gyandeeps
Copy link
Member

I like the proposal.

@nzakas
Copy link
Member

nzakas commented Jun 3, 2015

👎 I don't like having two ways of doing things, especially when the only reason for the change is to save typing 6 characters one time. We already match jshintConfig and jscsConfig, it seems that the convention has pretty much been set.

@gyandeeps
Copy link
Member

If its matches the styles used by other modules then I think it makes sense.

@sindresorhus
Copy link
Contributor Author

I don't like having two ways of doing things either, but you could always just document the eslint case, and have the old one as legacy. I know this is a long shot, but I'm looking for ways to make "annoying config I have to put into every package.json" look nicer and less verbose. I see no good reason the Config is there other than "the other project did it" fallacy.

@nzakas
Copy link
Member

nzakas commented Jun 3, 2015

I'm sorry, I don't find this line of reasoning to be convincing. "Look nicer" is far too subjective a reason to introduce redundancy. We will never be able to remove eslintConfig, as it's in use in too many places, so we would forever be stuck with two ways of doing the same thing.

@nzakas nzakas closed this as completed Jun 3, 2015
@eslint-deprecated eslint-deprecated bot locked and limited conversation to collaborators Feb 7, 2018
@eslint-deprecated eslint-deprecated bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Feb 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived due to age This issue has been archived; please open a new issue for any further discussion triage An ESLint team member will look at this issue soon
Projects
None yet
Development

No branches or pull requests

4 participants