Problem with prop 'env' in /test/unit/.eslintrc

[gaoupon]

since it is a avoriaz-jest-example, shall we change 'mocha' to 'jest'?

[eddyerburgh]

Yes definitely, are you able make a PR?

[gaoupon]

My pleasure.

[eddyerburgh]

Hey, I actually fixed this yesterday 🙂

…

On Tue, Aug 8, 2017 at 10:46 AM, 高鹏 ***@***.***> wrote: My pleasure. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AMlbW54sQ_kiZmmGq__-f7YVDOT6SoKRks5sWC6HgaJpZM4Ou66o> .

[gaoupon]

sorry for not seeing the message in time and I think the 'globals' prop is unneeded at the same file.
And in my opinion, a better and more flexible way to config eslint is to add "overrides" prop in the root/.eslintrc like this:

{
  ...,
  "overrides": [{
    "files": ["**/*.spec.js"],
    "env": {
      "jest": true
    }
  }]
}

In that way we can lint spec files not just in the test directory, but also any file in the project.