-
-
Notifications
You must be signed in to change notification settings - Fork 239
Conversation
@@ -0,0 +1 @@ | |||
package-lock = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is to prevent the creation of a package-lock.json file in the event someone uses npm i
"no-var": 0, | ||
"max-len": 0, | ||
"max-len": "off", | ||
"strict": "error", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed some files were using directives and some weren't, and I'm assuming we want to be in strict mode for all files.
} | ||
}, | ||
parserOptions: { | ||
sourceType: "script", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed to override https://github.com/babel/eslint-config-babel/blob/master/index.js#L7 for the strict
rule.
{ | ||
files: ["test/**/*"], | ||
env: { | ||
mocha: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Scoping this env just to tests so that we don't accidentally use these globals in the source code.
Update ESLint config
Looking to pick up #594 again and saw a few things that I thought should be updated in a separate PR.