Skip to content

Commit

Permalink
test to see if CI azure service bus somehow needs the double dash
Browse files Browse the repository at this point in the history
  • Loading branch information
Adam Ratzman committed Feb 25, 2025
1 parent ff77d7e commit d128b27
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions src/Aspire.Hosting/Dcp/DcpExecutor.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1023,7 +1023,7 @@ private async Task CreateExecutableAsync(AppResource er, ILogger resourceLogger,
// We still want to display the args in the dashboard so only add them to the custom arg annotations.
var annotationOnly = spec.ExecutionType == ExecutionType.IDE;

var launchProfileArgs = GetLaunchProfileArgs(project.GetEffectiveLaunchProfile()?.LaunchProfile, appHostArgs.Count > 0);
var launchProfileArgs = GetLaunchProfileArgs(project.GetEffectiveLaunchProfile()?.LaunchProfile/*, appHostArgs.Count > 0*/);
launchArgs.AddRange(launchProfileArgs.Select(a => (a, isSensitive: false, annotationOnly)));
}
}
Expand All @@ -1034,7 +1034,7 @@ private async Task CreateExecutableAsync(AppResource er, ILogger resourceLogger,
return launchArgs;
}

private static List<string> GetLaunchProfileArgs(LaunchProfile? launchProfile, bool containsAppHostArgs)
private static List<string> GetLaunchProfileArgs(LaunchProfile? launchProfile/*, bool containsAppHostArgs*/)
{
var args = new List<string>();
if (launchProfile is not null && !string.IsNullOrWhiteSpace(launchProfile.CommandLineArgs))
Expand All @@ -1043,11 +1043,10 @@ private static List<string> GetLaunchProfileArgs(LaunchProfile? launchProfile, b
if (cmdArgs.Count > 0)
{
// If there are no app host args, we don't need a redundant --
if (containsAppHostArgs)
{
//if (containsAppHostArgs)
//{
args.Add("--");
}

//}
args.AddRange(cmdArgs);
}
}
Expand Down

0 comments on commit d128b27

Please sign in to comment.