Closed backflip closed 7 years ago
I did use "extends": "eslint:recommended"
(http://eslint.org/docs/rules/) for another project. How would you see that? It sets some rules that we can then overwrite with ours. This produces quite a shorter .eslintrc
file.
I would also suggest to sort the rules alphabetically. This is as well how they are presented in their documentation and adds some easiness and expectation when reading the configuration.
@orioltf, alphabetical sorting added.
Do you want to go through the rules and compare them with eslint:recommended
? Or should we do this in a second step (ideally after having tested the auto-generated config in some real projects)? I kinda like the current, explicit config since it might help switching from JSHint/JSCS to ESLint by allowing you to compare the current with the previous configs.
.jsx
proved to be difficult to handle with JSHint.eslintrc
from previous.jshintrc
and.jscs
files (using https://www.npmjs.com/package/polyjuice). Minor adaptations where needed (see specific commit).var
declaration but separatingrequire
is not (yet) possible with ESLint (might be fixed in the future https://github.com/eslint/eslint/issues/6175). That's why I have combined these declarations in the.data.js
files. An alternative would be to disable theone-var
rule.// eslint-disable-line