Permalink
Browse files

Remove moot eslint-disable comments

  • Loading branch information...
sindresorhus committed May 7, 2017
1 parent 81d57a0 commit a1e08ab8fea773daab3489f05ad47fdd7f202644
Showing with 8 additions and 8 deletions.
  1. +1 βˆ’1 cli.js
  2. +1 βˆ’1 index.js
  3. +1 βˆ’1 lib/babel-config.js
  4. +1 βˆ’1 lib/test-worker.js
  5. +1 βˆ’1 test/concurrent.js
  6. +1 βˆ’1 test/sequence.js
  7. +2 βˆ’2 test/serialize-error.js
View
2 cli.js
@@ -12,7 +12,7 @@ const localCLI = resolveCwd('ava/cli');
// see https://github.com/nodejs/node/issues/6624
if (localCLI && path.relative(localCLI, __filename) !== '') {
debug('Using local install of AVA');
require(localCLI); // eslint-disable-line import/no-dynamic-require
require(localCLI);
} else {
if (debug.enabled) {
require('time-require'); // eslint-disable-line import/no-unassigned-import
View
@@ -2,7 +2,7 @@
// Ensure the same AVA install is loaded by the test file as by the test worker
if (process.env.AVA_PATH && process.env.AVA_PATH !== __dirname) {
module.exports = require(process.env.AVA_PATH); // eslint-disable-line import/no-dynamic-require
module.exports = require(process.env.AVA_PATH);
} else {
module.exports = require('./lib/main');
}
View
@@ -136,7 +136,7 @@ function build(projectDir, cacheDir, userOptions, powerAssert) {
return resolveOptions(baseConfig, cache, optionsFile, verifierFile);
})
.then(cacheKeys => ({
getOptions: require(optionsFile).getOptions, // eslint-disable-line import/no-dynamic-require
getOptions: require(optionsFile).getOptions,
// Include the seed in the cache keys used to store compilation results.
cacheKeys: Object.assign({seed}, cacheKeys)
}));
View
@@ -46,7 +46,7 @@ exports.setRunner = newRunner => {
runner = newRunner;
};
require(testPath); // eslint-disable-line import/no-dynamic-require
require(testPath);
// If AVA was not required, show an error
if (!runner) {
View
@@ -63,7 +63,7 @@ function passAsync(val) {
function failAsync(err) {
return {
run() {
return collect(Promise.resolve({ // eslint-disable-line promise/no-promise-in-callback
return collect(Promise.resolve({
passed: false,
reason: err
}));
View
@@ -56,7 +56,7 @@ function passAsync(val) {
function failAsync(err) {
return {
run() {
return collect(Promise.resolve({ // eslint-disable-line promise/no-promise-in-callback
return collect(Promise.resolve({
passed: false,
reason: err
}));
View
@@ -70,7 +70,7 @@ test('source file is an absolute path, after source map correction, even if alre
fs.writeFileSync(tmpFile + '.map', JSON.stringify(Object.assign(map, {sourceRoot}), null, 2));
try {
require(tmpFile).run(); // eslint-disable-line import/no-dynamic-require
require(tmpFile).run();
t.fail('Fixture should have thrown');
} catch (err) {
const serializedErr = serialize(err);
@@ -82,7 +82,7 @@ test('source file is an absolute path, after source map correction, even if alre
test('determines whether source file is within the project', t => {
const file = tempWrite.sync('module.exports = () => { throw new Error("hello") }');
try {
require(file)(); // eslint-disable-line import/no-dynamic-require
require(file)();
t.fail('Should have thrown');
} catch (err) {
const serializedErr = serialize(err);

0 comments on commit a1e08ab

Please sign in to comment.