Use new ESLint rule format (fixes #43) (#47)

Author: jfmengels

package.json

@@ -51,14 +51,14 @@
   "devDependencies": {
     "ava": "*",
     "coveralls": "^2.11.9",
-    "eslint": "^2.7.0",
+    "eslint": "^3.5.0",
     "eslint-ava-rule-tester": "^0.1.1",
     "nyc": "^6.4.0",
     "pify": "^2.3.0",
     "xo": "*"
   },
   "peerDependencies": {
-    "eslint": ">=2"
+    "eslint": ">=3"
   },
   "xo": {
     "esnext": true

rules/catch-error-name.js

@@ -23,7 +23,7 @@ function isLintablePromiseCatch(node) {
 	return arg0.type === 'FunctionExpression' || arg0.type === 'ArrowFunctionExpression';
 }
 
-module.exports = function (context) {
+const create = context => {
 	const opts = context.options[0];
 	const name = (opts && opts.name) || 'err';
 	const stack = [];
@@ -66,11 +66,18 @@ module.exports = function (context) {
 	};
 };
 
-module.exports.schema = [{
+const schema = [{
 	type: 'object',
 	properties: {
 		name: {
 			type: 'string'
 		}
 	}
 }];
+
+module.exports = {
+	create,
+	meta: {
+		schema
+	}
+};

rules/explicit-length-check.js

@@ -23,10 +23,15 @@ function checkExpression(context, node) {
 	}
 }
 
-module.exports = context => {
+const create = context => {
 	return {
 		IfStatement: node => {
 			checkExpression(context, node.test);
 		}
 	};
 };
+
+module.exports = {
+	create,
+	meta: {}
+};

rules/filename-case.js

@@ -66,7 +66,7 @@ function splitFilename(filename) {
 	};
 }
 
-module.exports = context => {
+const create = context => {
 	const chosenCase = cases[context.options[0].case || 'camelCase'];
 	const filenameWithExt = context.getFilename();
 
@@ -92,7 +92,7 @@ module.exports = context => {
 	};
 };
 
-module.exports.schema = [{
+const schema = [{
 	type: 'object',
 	properties: {
 		case: {
@@ -105,3 +105,10 @@ module.exports.schema = [{
 		}
 	}
 }];
+
+module.exports = {
+	create,
+	meta: {
+		schema
+	}
+};

rules/no-abusive-eslint-disable.js

@@ -2,7 +2,7 @@
 
 const disableRegex = /^eslint-disable(-next-line|-line)?($|(\s+([\w-]+))?)/;
 
-module.exports = context => ({
+const create = context => ({
 	Program: node => {
 		node.comments.forEach(comment => {
 			const value = comment.value.trim();
@@ -18,11 +18,16 @@ module.exports = context => ({
 						line: 0,
 						column: 0
 					},
-					// So specify it in the message
+					// So specify the location in the message
 					message: 'Specify the rules you want to disable at line {{line}}:{{column}}',
 					data: comment.loc.start
 				});
 			}
 		});
 	}
 });
+
+module.exports = {
+	create,
+	meta: {}
+};

rules/no-process-exit.js

@@ -1,5 +1,6 @@
 'use strict';
-module.exports = context => {
+
+const create = context => {
 	const startsWithHashBang = context.getSourceCode().lines[0].indexOf('#!') === 0;
 
 	if (startsWithHashBang) {
@@ -33,3 +34,8 @@ module.exports = context => {
 		}
 	};
 };
+
+module.exports = {
+	create,
+	meta: {}
+};

rules/throw-new-error.js

@@ -10,7 +10,7 @@ const errorTypes = [
 	'URIError'
 ];
 
-module.exports = context => ({
+const create = context => ({
 	ThrowStatement: node => {
 		const arg = node.argument;
 		const error = arg.callee;
@@ -24,3 +24,10 @@ module.exports = context => ({
 		}
 	}
 });
+
+module.exports = {
+	create,
+	meta: {
+		fixable: 'code'
+	}
+};