Merge pull request #148 from Workiva/support_ignore_for_packges_with_…

…executables_only_used_outside_lib FEDX-2722: Update the `ignore` field to ignore everything, not just specific checks
Workiva · Mar 6, 2025 · f47c8d7 · f47c8d7
2 parents 16beb4d + b7d4e69
commit f47c8d7
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/lib/src/dependency_validator.dart b/lib/src/dependency_validator.dart
@@ -263,7 +263,8 @@ Future<void> run() async {
           .difference(packagesUsedInPublicFiles)
           .difference(packagesUsedOutsidePublicDirs)
         // Remove this package, since we know they're using our executable
-        ..remove(dependencyValidatorPackageName);
+        ..remove(dependencyValidatorPackageName)
+        ..removeAll(ignoredPackages);
 
   final packageConfig = await findPackageConfig(Directory.current);
   if (packageConfig == null) {
@@ -338,9 +339,6 @@ Future<void> run() async {
     );
   }
 
-  // Ignore known unused packages
-  unusedDependencies.removeAll(ignoredPackages);
-
   if (unusedDependencies.isNotEmpty) {
     log(
       Level.WARNING,