Skip to content

Commit 6563f0a

Browse files
authored
Merge pull request #124 from Sakeeb91/issue-116-run-arguments
Fix run command argument builder
2 parents 4216ede + 1a7182c commit 6563f0a

1 file changed

Lines changed: 8 additions & 7 deletions

File tree

src/FlowCtl/Commands/Run/RunCommandOptionsHandler.cs

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -65,14 +65,15 @@ private Task Execute(RunCommandOptions options)
6565
return Task.CompletedTask;
6666
}
6767

68+
/// <summary>
69+
/// Builds the argument string that is forwarded to the FlowSynx engine.
70+
/// </summary>
71+
/// <param name="options">The run command options selected by the user.</param>
72+
/// <returns>A space-delimited argument string.</returns>
6873
private string GetArgumentStr(RunCommandOptions options)
6974
{
70-
var argList = new List<string>
71-
{
72-
"--start"
73-
};
74-
75-
return argList.Count == 0 ? string.Empty : string.Join(' ', argList);
75+
var arguments = new List<string> { "--start" };
76+
return string.Join(' ', arguments);
7677
}
7778

7879
private void OutputDataHandler(object sendingProcess, DataReceivedEventArgs outLine)
@@ -84,4 +85,4 @@ private void ErrorDataHandler(object sendingProcess, DataReceivedEventArgs outLi
8485
{
8586
if (outLine.Data != null) _flowCtlLogger.WriteError(outLine.Data);
8687
}
87-
}
88+
}

0 commit comments

Comments
 (0)