Skip to content

Commit

Permalink
Resolve deprecation warning message from eslint while running eslint (#…
Browse files Browse the repository at this point in the history
…1586)

* Resolve eslint deprecation warning

  There was a DeprecationWarning message from eslint saying that `ecmaFeatures` property has 
  been deprecated.
  Moved it under the `parserOptions` as per recommended here - 
  https://eslint.org/docs/user-guide/migrating-to-2.0.0.
* Set escmaVersion = 6
* Use ES6 built-in global variables
* Remove flags in favor of ecmaVersion
  • Loading branch information
kabirbaidhya authored and nknapp committed Oct 28, 2019
1 parent b8913fc commit 7052e88
Showing 1 changed file with 6 additions and 18 deletions.
24 changes: 6 additions & 18 deletions .eslintrc.js
Expand Up @@ -4,22 +4,8 @@ module.exports = {
"self": false
},
"env": {
"node": true
},
"ecmaFeatures": {
// Enabling features that can be implemented without polyfills. Want to avoid polyfills at this time.
"arrowFunctions": true,
"blockBindings": true,
"defaultParams": true,
"destructuring": true,
"modules": true,
"objectLiteralComputedProperties": true,
"objectLiteralDuplicateProperties": true,
"objectLiteralShorthandMethods": true,
"objectLiteralShorthandProperties": true,
"restParams": true,
"spread": true,
"templateStrings": true
"node": true,
"es6": true
},
"rules": {
// overrides eslint:recommended defaults
Expand Down Expand Up @@ -124,6 +110,8 @@ module.exports = {
"no-var": "warn"
},
"parserOptions": {
"sourceType": "module"
"sourceType": "module",
"ecmaVersion": 6,
"ecmaFeatures": {}
}
}
}

0 comments on commit 7052e88

Please sign in to comment.