fix(linter): wildcard sourceTag should be respected for banned imports (#14418)

meeroslav · web-flow · commit 2b239634156f · 2023-01-17T14:59:21.000+01:00
diff --git a/packages/eslint-plugin-nx/src/utils/runtime-lint-utils.ts b/packages/eslint-plugin-nx/src/utils/runtime-lint-utils.ts
@@ -244,7 +244,7 @@ export function hasBannedImport(
       tags = [c.sourceTag];
     }
 
-    return tags.every((t) => (source.data.tags || []).includes(t));
+    return tags.every((t) => hasTag(source, t));
   });
   return depConstraints.find((constraint) =>
     isConstraintBanningProject(target, constraint)

Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ export function hasBannedImport(`
`244`	`244`	`tags = [c.sourceTag];`
`245`	`245`	`}`
`246`	`246`
`247`		`- return tags.every((t) => (source.data.tags \|\| []).includes(t));`
	`247`	`+ return tags.every((t) => hasTag(source, t));`
`248`	`248`	`});`
`249`	`249`	`return depConstraints.find((constraint) =>`
`250`	`250`	`isConstraintBanningProject(target, constraint)`