Skip to content

Commit

Permalink
Merge pull request #57 from NA-Wen/killtestprocess_windows
Browse files Browse the repository at this point in the history
make sure test process can be killed on windows
  • Loading branch information
thinkwee authored Sep 14, 2023
2 parents 1245025 + 0ad26e3 commit ea3d415
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions chatdev/chat_env.py
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,8 @@ def exist_bugs(self) -> tuple[bool, str]:
os.killpg(os.getpgid(process.pid), signal.SIGTERM)
else:
os.kill(process.pid, signal.SIGTERM)
if process.poll() is None:
os.kill(process.pid,signal.CTRL_BREAK_EVENT)

if return_code == 0:
return False, success_info
Expand Down

0 comments on commit ea3d415

Please sign in to comment.