Updates (#220)

Author: novemberborn

.travis.yml

@@ -1,9 +1,8 @@
 language: node_js
 node_js:
-  - '10'
-  - '8'
-  - '6'
-before_install:
-  - npm install --global npm
-after_success:
-  - './node_modules/.bin/nyc report --reporter=text-lcov | ./node_modules/.bin/coveralls'
+  - 11
+  - 10
+  - 8
+  - 6
+before_install: npm install --global npm
+after_success: npx codecov --file=./coverage/lcov.info

package.json

@@ -43,18 +43,22 @@
 	"devDependencies": {
 		"ava": "^1.0.1",
 		"babel-eslint": "^10.0.1",
-		"coveralls": "^3.0.2",
+		"codecov": "^3.2.0",
 		"eslint": "^5.12.0",
 		"eslint-ava-rule-tester": "^3.0.0",
 		"js-combinatorics": "^0.5.4",
 		"nyc": "^13.1.0",
 		"pify": "^4.0.1",
-		"xo": "^0.23.0"
+		"xo": "^0.24.0"
 	},
 	"peerDependencies": {
 		"eslint": ">=5.9.0"
 	},
-	"ava": {
-		"failWithoutAssertions": true
+	"nyc": {
+		"reporter": [
+			"html",
+			"lcov",
+			"text"
+		]
 	}
 }

rules/no-async-fn-without-await.js

@@ -42,6 +42,7 @@ const create = context => {
 					message: 'Function was declared as `async` but doesn\'t use `await`'
 				});
 			}
+
 			asyncTest = null;
 			testUsed = false;
 		})

rules/no-duplicate-modifiers.js

@@ -36,6 +36,7 @@ const create = context => {
 						message: `Duplicate test modifier \`${current.name}\`.`
 					});
 				}
+
 				return current;
 			});
 		})

test/fixtures/load-config/no-default-export/ava.config.js

@@ -1 +1 @@
-export const config = {}; // eslint-disable-line import/prefer-default-export
+export const config = {};

util.js

@@ -110,15 +110,6 @@ exports.getTestModifier = (node, mod) => {
 	return getTestModifiers(node).find(property => property.name === mod);
 };
 
-/**
- * Removes given test-modifier from the source surrounding the given node
- *
- * @param {string} params.modifier - Name of the modifier
- * @param {Node} params.node - ESTree-node as provided by ESLint
- * @param {Context} params.context - ESLint-context as provided
- *
- * @return {Array} Compound parameters to be used as arguments for `fix.replaceTextRange()`
- */
 exports.removeTestModifier = params => {
 	const modifier = params.modifier.trim();
 	const range = exports.getTestModifier(params.node, modifier).range.slice();
@@ -128,6 +119,7 @@ exports.removeTestModifier = params => {
 	while (source.charAt(dotPosition) !== '.') {
 		dotPosition -= 1;
 	}
+
 	let snippet = source.slice(dotPosition, range[1]);
 	snippet = snippet.replace(replacementRegExp, '');
 	return [[dotPosition, range[1]], snippet];
@@ -146,17 +138,13 @@ const getMembers = node => {
 exports.getMembers = getMembers;
 
 const repoUrl = 'https://github.com/avajs/eslint-plugin-ava';
-/**
- * Return the URL of the rule's documentation, either from parameter or the
- * requiring file's name.
- * @param  {String} ruleName The name of the rule to generate a URL for.
- * @return {String}          The URL of the rule's documentation.
- */
+
 const getDocsUrl = (filename, commitHash) => {
 	const ruleName = path.basename(filename, '.js');
 	commitHash = commitHash || `v${pkg.version}`;
 	return `${repoUrl}/blob/${commitHash}/docs/rules/${ruleName}.md`;
 };
+
 exports.getDocsUrl = getDocsUrl;
 
 const assertionMethodsNumArguments = new Map([