Improve error experience for Restore command without a project (#45129)

dotnet · Nov 26, 2024 · 208df00 · 208df00
2 parents 5e1bc3c + 12a8505
commit 208df00
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Cli/dotnet/commands/dotnet-workload/restore/WorkloadRestoreCommand.cs b/src/Cli/dotnet/commands/dotnet-workload/restore/WorkloadRestoreCommand.cs
@@ -50,10 +50,12 @@ public override int Execute()
 
             recorder.Run(() =>
             {
-                // First update manifests and install a workload set as necessary
+                // First discover projects. This may return an error if no projects are found, and we shouldn't delay until after Update if that's the case.
+                var allProjects = DiscoverAllProjects(Directory.GetCurrentDirectory(), _slnOrProjectArgument).Distinct();
+
+                // Then update manifests and install a workload set as necessary
                 new WorkloadUpdateCommand(_result, recorder: recorder, isRestoring: true).Execute();
 
-                var allProjects = DiscoverAllProjects(Directory.GetCurrentDirectory(), _slnOrProjectArgument).Distinct();
                 List<WorkloadId> allWorkloadId = RunTargetToGetWorkloadIds(allProjects);
                 Reporter.WriteLine(string.Format(LocalizableStrings.InstallingWorkloads, string.Join(" ", allWorkloadId)));