Skip to content

Commit

Permalink
fix: remove signals.info shorthand because linux doesn't have SIGINFO
Browse files Browse the repository at this point in the history
  • Loading branch information
kt3k committed Jan 23, 2020
1 parent fe55b49 commit 79f5762
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 18 deletions.
3 changes: 0 additions & 3 deletions cli/js/lib.deno_runtime.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2178,9 +2178,6 @@ declare namespace Deno {
/** Returns the stream of SIGHUP signals.
* This method is the shorthand for Deno.signal(Deno.Signal.SIGHUP). */
hungup: () => SignalStream;
/** Returns the stream of SIGINFO signals.
* This method is the shorthand for Deno.signal(Deno.Signal.SIGINFO). */
info: () => SignalStream;
/** Returns the stream of SIGINT signals.
* This method is the shorthand for Deno.signal(Deno.Signal.SIGINT). */
interrupt: () => SignalStream;
Expand Down
10 changes: 0 additions & 10 deletions cli/js/signal_test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,13 +39,6 @@ if (Deno.build.os === "win") {
Error,
"not implemented"
);
assertThrows(
() => {
Deno.signals.info(); // for SIGINFO
},
Error,
"not implemented"
);
assertThrows(
() => {
Deno.signals.interrupt(); // for SIGINT
Expand Down Expand Up @@ -160,9 +153,6 @@ if (Deno.build.os === "win") {
s = Deno.signals.hungup(); // for SIGHUP
assert(s instanceof Deno.SignalStream);
s.dispose();
s = Deno.signals.info(); // for SIGINFO
assert(s instanceof Deno.SignalStream);
s.dispose();
s = Deno.signals.interrupt(); // for SIGINT
assert(s instanceof Deno.SignalStream);
s.dispose();
Expand Down
5 changes: 0 additions & 5 deletions cli/js/signals.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,11 +50,6 @@ export const signals = {
hungup(): SignalStream {
return createSignalStream(Signal.SIGHUP);
},
/** Returns the stream of SIGINFO signals.
* This method is the shorthand for Deno.signal(Deno.Signal.SIGINFO). */
info(): SignalStream {
return createSignalStream(Signal.SIGINFO);
},
/** Returns the stream of SIGINT signals.
* This method is the shorthand for Deno.signal(Deno.Signal.SIGINT). */
interrupt(): SignalStream {
Expand Down

0 comments on commit 79f5762

Please sign in to comment.