updates for eslint cleanup

[mikekidder]

Notes: Minor changes to code in async, real-world examples to clear the eslint errors.

[hzoo]

maybe just disable the no-console rule?

[mikekidder]

could if needed elsewhere. just the three lines needed it in one file. but do agree, this is source.. not examples

[gaearon]

Is linting no longer running as part of Travis build? Why did we not see these lint errors?

[mikekidder]

I looked at history of .travis.yml, don't believe linting was ever part of it?

[gaearon]

Lol, maybe so. Would you mind adding it?

[hzoo]

Might want to update to eslint 1.2.1 and babel-eslint to 4.0.10 as well (although that could be another PR)?

[gaearon]

@hzoo If you would like to submit a separate PR updating them, please feel free to! I haven't used eslint 1.x yet.

[mikekidder]

How about @hzoo adds lint to Travis with that PR. I am getting an error running "npm run lint" under windows, even for the master branch. Throwing an exception in \eslint\lib\config-validator.js, so its going to need to be looked at some more before adding to Travis CI.

[mikekidder]

... yeah, its saying this:

Configuration for rule "generator-star-spacing" is invalid:
Value "[object Object]" must be an enum value.

UPDATE: And its not coming from redux's .eslintrc, but a dependency for package "is-nan"
babel-core -> is-nan -> define-properties

n/a to this PR

[mikekidder]

Successful linting in Travis CI :)

[mikekidder]

@hzoo I also have changes ready to go with the following npm updates:
babel-eslint 4.1.0, eslint 1.2.1, eslint-plugin-react 3.2.3, eslint-config-airbnb 0.0.8

If you are not working on a PR for it, I will submit. Let me know!

[hzoo]

I haven't started it yet so go ahead!