Skip to content

Commit 2e23c25

Browse files
authored
Merge pull request #488 from toxik/fix/script-task
Fix the `ScriptProcessExecutor` to honor the `await` property
2 parents 9a5fc4d + b780dc1 commit 2e23c25

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/runner/Synapse.Runner/Services/Executors/ScriptProcessExecutor.cs

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright © 2024-Present The Synapse Authors
1+
// Copyright © 2024-Present The Synapse Authors
22
//
33
// Licensed under the Apache License, Version 2.0 (the "License"),
44
// you may not use this file except in compliance with the License.
@@ -69,7 +69,7 @@ protected override async Task DoExecuteAsync(CancellationToken cancellationToken
6969
? null
7070
: await this.ProcessDefinition.Environment.ToAsyncEnumerable().ToDictionaryAwaitAsync(kvp => ValueTask.FromResult(kvp.Key), async kvp => (await this.EvaluateAndSerializeAsync(kvp.Value, cancellationToken).ConfigureAwait(false))!, cancellationToken).ConfigureAwait(false);
7171
var process = await executor.ExecuteAsync(script, arguments, environment, cancellationToken).ConfigureAwait(false);
72-
if (this.Task.Definition.Run.Await != false)
72+
if (this.Task.Definition.Run.Await == false)
7373
{
7474
await this.SetResultAsync(new(), this.Task.Definition.Then, cancellationToken).ConfigureAwait(false);
7575
return;

0 commit comments

Comments
 (0)