Skip to content

Commit

Permalink
Moved "Installing Dependencies" message to only print if there are de…
Browse files Browse the repository at this point in the history
…pendencies to install. (#1851)
  • Loading branch information
jedieaston authored Jan 13, 2022
1 parent 5f3f9d4 commit ae586be
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/AppInstallerCLICore/Workflows/DependenciesFlow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -155,8 +155,6 @@ namespace AppInstaller::CLI::Workflow
return;
}

info << Resource::String::DependenciesFlowInstall << std::endl;

context << OpenDependencySource;
if (context.IsTerminated())
{
Expand Down Expand Up @@ -238,6 +236,11 @@ namespace AppInstaller::CLI::Workflow
}
}

if (!dependencyPackageContexts.empty())
{
info << Resource::String::DependenciesFlowInstall << std::endl;
}

// Install dependencies in the correct order
context.Add<Execution::Data::PackagesToInstall>(std::move(dependencyPackageContexts));
context << Workflow::InstallMultiplePackages(m_dependencyReportMessage, APPINSTALLER_CLI_ERROR_INSTALL_DEPENDENCIES, {}, false, true);
Expand Down

0 comments on commit ae586be

Please sign in to comment.