fix: logged message about installing packages even though nothing was…

… installed
Zeugwerk · Sep 17, 2024 · 562c9dd · 562c9dd
1 parent d218b11
commit 562c9dd
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/TwinpackCore/Core/TwinpackService.cs b/TwinpackCore/Core/TwinpackService.cs
@@ -482,12 +482,14 @@ public async System.Threading.Tasks.Task<List<PackageItem>> AddPackagesAsync(Lis
             var downloadedPackageVersions = await downloadPackagesTask;
 
             // install downloaded packages
-            foreach (var package in downloadedPackageVersions)
+            if(_automationInterface != null && (options?.SkipInstall == null || options?.SkipInstall == false))
             {
-                _logger.Info($"Installing {package.PackageVersion.Name} {package.PackageVersion.Version}");
-                if(_automationInterface != null && (options?.SkipInstall == null || options?.SkipInstall == false))
-                    await _automationInterface.InstallPackageAsync(package, cachePath: downloadPath);
-                cancellationToken.ThrowIfCancellationRequested();
+                foreach (var package in downloadedPackageVersions)
+                {
+                    _logger.Info($"Installing {package.PackageVersion.Name} {package.PackageVersion.Version}");
+                    await _automationInterface.InstallPackageAsync(package, cachePath: downloadPath); 
+                    cancellationToken.ThrowIfCancellationRequested();
+                }
             }
 
             // add affected packages as references