Explorar el Código

Avoid processing packages in parallel in ValidateApiDiff (#19511)

Julien Lebosquain hace 2 meses
padre
commit
0225edd8c0
Se han modificado 1 ficheros con 6 adiciones y 13 borrados
  1. 6 13
      nukebuild/ApiDiffHelper.cs

+ 6 - 13
nukebuild/ApiDiffHelper.cs

@@ -47,20 +47,13 @@ public static class ApiDiffHelper
 
         var allErrors = new List<string>();
 
-        Parallel.ForEach(
-            packageDiff.Frameworks,
-            framework =>
-            {
-                var args = $""" -l="{framework.BaselineFolderPath}" -r="{framework.CurrentFolderPath}" {suppressionArgs}""";
-
-                var localErrors = GetErrors(apiCompatTool(args));
+        foreach (var framework in packageDiff.Frameworks)
+        {
+            var args = $""" -l="{framework.BaselineFolderPath}" -r="{framework.CurrentFolderPath}" {suppressionArgs}""";
 
-                if (localErrors.Length > 0)
-                {
-                    lock (allErrors)
-                        allErrors.AddRange(localErrors);
-                }
-            });
+            var localErrors = GetErrors(apiCompatTool(args));
+            allErrors.AddRange(localErrors);
+        }
 
         ThrowOnErrors(allErrors, packageDiff.PackageId, "ValidateApiDiff");
     }