Skip to content

Commit 38c6b63

Browse files
branchseerclaude
andcommitted
fix(pty_terminal): use yield_now instead of sleep, revert temp CI
Replace disallowed std::thread::sleep with std::thread::yield_now in the signal polling loop. Remove temporary 10x retry CI step (all 10 retries passed, confirming stability). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent c539ee7 commit 38c6b63

File tree

2 files changed

+1
-8
lines changed

2 files changed

+1
-8
lines changed

.github/workflows/ci.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -173,13 +173,6 @@ jobs:
173173
pnpm install
174174
175175
- run: cargo test
176-
# TEMPORARY: verify send_ctrl_c_interrupts_process stability (10 retries)
177-
- name: Retry ctrl_c test 10 times
178-
run: |
179-
for i in $(seq 1 10); do
180-
echo "--- Run $i/10 ---"
181-
cargo test -p pty_terminal --test terminal -- send_ctrl_c_interrupts_process
182-
done
183176

184177
fmt:
185178
name: Format and Check Deps

crates/pty_terminal/tests/terminal.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -309,7 +309,7 @@ fn send_ctrl_c_interrupts_process() {
309309
stdout().flush().unwrap();
310310
std::process::exit(0);
311311
}
312-
std::thread::sleep(std::time::Duration::from_millis(10));
312+
std::thread::yield_now();
313313
}
314314
}
315315

0 commit comments

Comments
 (0)