This repository has been archived by the owner on Nov 27, 2019. It is now read-only.

Fix TODOs in .eslintrc.yml #585

Open

pdehaan opened this issue Sep 21, 2016 · 1 comment

Contributor

pdehaan commented Sep 21, 2016 •

edited

Loading

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?

This was referenced Sep 21, 2016

chore: Add ESLint no-console and max-len rules #587

Merged

chore: Switch from JSHint/JSCS to ESLint #581

Merged

Contributor Author

pdehaan commented Sep 22, 2016

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.