Skip to content

Commit 2c6d40c

Browse files
committed
fix Windows replacement for process kill during deployment
The old Windows rule matched 'Exit code: [KILLED]' which was never present because 'Exit code: 1' (Windows exit code for kill) was never normalized to '[KILLED]' -- the [KILLED] normalization only fires via exit code 137 (Linux) or after [PROCESS_KILLED] is already inserted. Match 'Exit code: 1' directly (the raw Windows exit code), then insert [PROCESS_KILLED] and normalize in one step. Co-authored-by: Isaac
1 parent f628251 commit 2c6d40c

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

acceptance/bundle/deploy/wal/test.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@ Old = '(\[PROCESS_KILLED\]\n\nExit code: )1'
3030
New = '${1}[KILLED]'
3131

3232
# On Windows, no bash "Killed" message appears when CLI has produced output before termination.
33-
# Insert [PROCESS_KILLED] between last output line and exit code for consistency.
33+
# Match the raw exit code 1 (Windows never gets 137 or [PROCESS_KILLED] marker first).
3434
[[Repls]]
35-
Old = '(Deploying resources\.\.\.)\n\nExit code: \[KILLED\]'
35+
Old = '(Deploying resources\.\.\.)\n\nExit code: 1'
3636
New = """${1}
3737
[PROCESS_KILLED]
3838

0 commit comments

Comments
 (0)