fix: enable nocomment glob option for rule no-unassigned-import's allow option (#352)

Author: JounQin

.changeset/wise-tables-stop.md

@@ -0,0 +1,5 @@
+---
+"eslint-plugin-import-x": patch
+---
+
+fix: enable `nocomment` glob option for rule `no-unassigned-import`'s `allow` option

src/rules/no-unassigned-import.ts

@@ -21,8 +21,11 @@ function testIsAllow(
 
   return globs.some(
     glob =>
-      minimatch(filePath, glob) ||
-      minimatch(filePath, path.resolve(glob), { windowsPathsNoEscape: true }),
+      minimatch(filePath, glob, { nocomment: true }) ||
+      minimatch(filePath, path.resolve(glob), {
+        nocomment: true,
+        windowsPathsNoEscape: true,
+      }),
   )
 }
 

test/rules/no-unassigned-import.spec.ts

@@ -73,6 +73,20 @@ ruleTester.run('no-unassigned-import', rule, {
       filename: path.resolve('src/app.js'),
       options: [{ allow: ['src/styles/**', '**/scripts/*.js'] }],
     }),
+    tValid({
+      code: 'import "#app/side-effects.js"',
+      // exactly matched
+      options: [{ allow: ['#app/side-effects.js'] }],
+    }),
+    tValid({
+      code: 'import "#app/side-effects.js"',
+      /**
+       * Matched by glob
+       *
+       * @see https://github.com/isaacs/minimatch#comparisons-to-other-fnmatchglob-implementations
+       */
+      options: [{ allow: [String.raw`\#app/side-effects.js`] }],
+    }),
   ],
   invalid: [
     tInvalid({