actually set parserOptions for babel-eslint because of eslint/eslint#…

…8744
lukekarrys · Jun 23, 2017 · e3d7cd8 · e3d7cd8
1 parent 30c7517
commit e3d7cd8
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 7 deletions.
diff --git a/base.js b/base.js
@@ -1,3 +1,5 @@
+'use strict';
+
 module.exports = {
   env: {
     browser: false,

diff --git a/index.js b/index.js
@@ -1,8 +1,14 @@
+'use strict';
+
 module.exports = {
   'extends': './base.js',
 
   parser: 'babel-eslint',
 
+  parserOptions: {
+    sourceType: 'module'
+  },
+
   plugins: [
     'react',
     'classes',

diff --git a/no-babel.js b/no-babel.js
@@ -1,3 +1,5 @@
+'use strict';
+
 module.exports = {
   'extends': './base.js',
 

diff --git a/package.json b/package.json
@@ -18,7 +18,6 @@
     "eslint-plugin-react": "^7.1.0",
     "espree": "^3.4.3",
     "git-validate": "^2.2.2",
-    "greenkeeper-postpublish": "^1.0.1",
     "tape": "^4.6.3",
     "webpack": "^3.0.0"
   },
@@ -43,8 +42,7 @@
     "url": "git://github.com/lukekarrys/eslint-config.git"
   },
   "scripts": {
-    "lint": "eslint -c index.js .",
-    "postpublish": "greenkeeper-postpublish",
+    "lint": "eslint -c no-babel.js .",
     "test": "tape test/*.js",
     "validate": "npm ls >/dev/null"
   }

diff --git a/test/validate-config.js b/test/validate-config.js
@@ -28,10 +28,7 @@ t(bar);
 test('load config in eslint to validate all rule syntax is correct', (t) => {
   const {results, errorCount, warningCount} = new CLIEngine({
     useEslintrc: false,
-    configFile: 'index.js',
-    parserOptions: {
-      sourceType: 'module'
-    }
+    configFile: 'index.js'
   }).executeOnText(code);
 
   t.equal(results.length, 1, 'One result (empty)');