Skip to content
This repository was archived by the owner on May 29, 2023. It is now read-only.

Commit 8d0fda9

Browse files
committed
[eslint config] [base] Add comments on new import rules.
1 parent b59bfff commit 8d0fda9

File tree

1 file changed

+5
-2
lines changed
  • packages/eslint-config-airbnb-base/rules

1 file changed

+5
-2
lines changed

packages/eslint-config-airbnb-base/rules/imports.js

+5-2
Original file line numberDiff line numberDiff line change
@@ -155,9 +155,11 @@ module.exports = {
155155
allow: [],
156156
}],
157157

158-
// Warn if a module could be mistakenly parsed as a script by a consumer leveraging Unambiguous
159-
// JavaScript Grammar
158+
// Warn if a module could be mistakenly parsed as a script by a consumer
159+
// leveraging Unambiguous JavaScript Grammar
160160
// https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/unambiguous.md
161+
// this should not be enabled until this proposal has at least been *presented* to TC39.
162+
// At the moment, it's not a thing.
161163
'import/unambiguous': 'off',
162164

163165
// Forbid Webpack loader syntax in imports
@@ -166,6 +168,7 @@ module.exports = {
166168

167169
// Prevent unassigned imports
168170
// https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unassigned-import.md
171+
// importing for side effects is perfectly acceptable, if you need side effects.
169172
'import/no-unassigned-import': 'off',
170173
},
171174
};

0 commit comments

Comments
 (0)