move check to the top

microsoft · Oct 18, 2023 · f464f31 · f464f31
1 parent c33686e
commit f464f31
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/AppInstallerCLICore/Workflows/DependenciesFlow.cpp b/src/AppInstallerCLICore/Workflows/DependenciesFlow.cpp
@@ -240,6 +240,11 @@ namespace AppInstaller::CLI::Workflow
 
     void CreateDependencySubContexts::operator()(Execution::Context& context) const
     {
+        if (context.Args.Contains(Execution::Args::Type::SkipDependencies))
+        {
+			return;
+		}
+
         auto info = context.Reporter.Info();
         auto error = context.Reporter.Error();
         const auto& rootManifest = context.Get<Execution::Data::Manifest>();
@@ -249,9 +254,9 @@ namespace AppInstaller::CLI::Workflow
         const auto& rootInstaller = context.Get<Execution::Data::Installer>();
         const auto& rootDependencies = rootInstaller->Dependencies;
 
-        if (rootDependencies.Empty() || context.Args.Contains(Execution::Args::Type::SkipDependencies))
+        if (rootDependencies.Empty())
         {
-            // If there's no dependencies or user skips them, there's nothing to do aside of logging the outcome
+            // If there's no dependencies, there's nothing to do aside of logging the outcome
             return;
         }