replace jshint + jscs by eslint

[MoOx]

Related to #101 with babel-eslint

[jonathanKingston]

I would be interested in helping pick this up to get me into the source unless you are stuck into it? (Just trying to find a way to cut in and help)

[MoOx]

This issue is somehow related to #101

Here is the eslint file I am using

---
# babel support more syntax stuff than eslint for now
parser: babel-eslint

ecmaFeatures:
  modules: true

env:
  es6: true
  browser: true
  node: true

# 0: off, 1: warning, 2: error
rules:
  # semicolons are useless
  semi: [2, "never"]

  quotes: [2, "double"]

  # 2 spaces indentation
  indent: [2, 2]

  # trailing coma are cool for diff
  comma-dangle: [2, "always-multiline"]

  # enforce comma at eol (never before)
  comma-style: [2, "last"]

You can take it if you want, cssnext is not that huge (only index.js < 200 lines + test/*)

[jonathanKingston]

Cool yeah I wasn't expecting it to be hard but I just wanted something to work on.

I'd like to get to being as explicit as this too: https://github.com/jonathanKingston/ember-cli-eslint/blob/master/.eslintrc

[MoOx]

Your file override lots of default, I like to just define what is modified from default values (most default values of eslint are good enough).

[jonathanKingston]

@MoOx certainly minimum pass is the best. I mostly like to be explicit also just in case the defaults change (yes I am paranoid mostly 😄 ).

Anyway I'll have a go with your set and see if there are any others that might be relevant to enable.

[jonathanKingston]

@MoOx having some trouble getting the tests in master to pass along with the tests here in the work I have done. Looks like a timeout issue rather than an actual issue.
It passes the ESLint rules you specified and no need for babel-eslint with the current code.

[MoOx]

Can you rebase on master (I "fixed" (skipped some) tests for CI) and tell me exactly where is the issue ?

[MoOx]

This is good.