search

denoland / deno

A modern runtime for JavaScript and TypeScript.
https://deno.com
MIT License
97.46k stars 5.37k forks source link

suggestion: deprecate `Deno.kill()` #25405

Closed iuioiua closed 2 months ago

iuioiua commented 2 months ago

IIUC, it has no other use case outside spawned processes. After Deno 2, one can just use Deno.ChildProcess.prototype.kill. At this point, the deprecation can happen at Deno 2, and the removal sometime after.

lucacasonato commented 2 months ago

Deno.kill is useful to kill process groups by passing in a negative process id. This is not possible with ChildProcess.prototype.kill.