Skip to content

Commit 393e8e1

Browse files
garykingsindresorhus
authored andcommitted
import-index: Fix for require() (#436)
1 parent 4d5f5cb commit 393e8e1

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

rules/import-index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const isImportingIndex = value => regexp.test(value);
66
const normalize = value => value.replace(regexp, '$1');
77

88
const importIndex = (context, node, argument) => {
9-
if (isImportingIndex(argument.value)) {
9+
if (argument && isImportingIndex(argument.value)) {
1010
context.report({
1111
node,
1212
message: 'Do not reference the index file directly.',

test/import-index.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ const error = {
1818

1919
ruleTester.run('import-index', rule, {
2020
valid: [
21+
'const m = require()',
2122
'const m = require(\'.\')',
2223
'const m = require(\'..\')',
2324
'const m = require(\'../..\')',

0 commit comments

Comments
 (0)