Fix TODOs in .eslintrc.yml

mozilla-jetpack / jpm

Jetpack Manager for Node.js

https://www.npmjs.org/package/jpm

Mozilla Public License 2.0

164 stars 75 forks source link

Fix TODOs in .eslintrc.yml #585

Open pdehaan opened 8 years ago

pdehaan commented 8 years ago

The current JSHint/JSCS => ESLint added a few TODOs to re-evaluate rules. Now that we've landed the ESLint patch, we should reconsider the rules.

$ cat .eslintrc.yml | grep -i TODO

  brace-style: off   # TODO: Set to warn|error?
  comma-dangle: off  # TODO: Set to `[error, always|never]`?
  curly: off         # TODO: Set to error?

pdehaan commented 8 years ago

Fixed max-len and no-console in PR #587.