Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove eslint dependency #2

Closed
ruiquelhas opened this issue Nov 2, 2015 · 1 comment
Closed

Remove eslint dependency #2

ruiquelhas opened this issue Nov 2, 2015 · 1 comment

Comments

@ruiquelhas
Copy link
Contributor

eslint is currently required as a dependency because it's being used as a linter for the module itself since it has additional code to load the rules declared in yaml.

If the rules are expressed in a regular Node.js™ module using module.exports and the proper main file path in package.json, the dependency is no longer needed.

@ruimarinho
Copy link
Contributor

👍 for this. We should look into the maintainers own config - https://www.npmjs.com/package/eslint-config-eslint.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants