feat(setRaw): Remove the need for unstable flag by upgrading dependencies and min deno version (#13)

upgrade dependencies

Author: sylc

.vscode/settings.json

@@ -1,7 +1,6 @@
 {
   "deno.enable": true,
   "deno.lint": true,
-  "deno.unstable": true,
   "deno.suggest.imports.hosts": {
     "https://deno.land": true
   }

README.md

@@ -14,7 +14,7 @@
 ### Install
 
 ```
-deno install -f --unstable --allow-run --allow-net https://deno.land/x/dkill@0.7.0/cli.ts
+deno install -f --allow-run --allow-net https://deno.land/x/dkill@0.7.0/cli.ts
 ```
 
 You can then use it using command `dkill`

cli.test.ts

@@ -8,7 +8,7 @@ Deno.test("killing by pid", async () => {
 
   // call dkill
   const pDkill = Deno.run({
-    cmd: ["deno", "run", "-A", "--unstable", "./cli.ts", `${pTest.pid}`],
+    cmd: ["deno", "run", "-A", "./cli.ts", `${pTest.pid}`],
   });
   // wait dkill finishes
   await pDkill.status();
@@ -22,28 +22,37 @@ Deno.test("killing by pid", async () => {
 
   assertNotEquals(status.code, 0);
 });
-Deno.test("killing by port", async () => {
+Deno.test("killing by ports", async () => {
   // create a webserver
-  const pTest = Deno.run({
-    cmd: ["deno", "run", "-A", "--unstable", "./src/tests/utils.ts"],
+  const pTest1 = Deno.run({
+    cmd: ["deno", "run", "-A", "./src/tests/utils.ts"],
+  });
+  const pTest2 = Deno.run({
+    cmd: ["deno", "run", "-A", "./src/tests/utils.ts", "8081"],
   });
 
+
   // give time fo the webserver to start and the port be discoverable
   await delay(5000);
 
   // call dkill
   const pDkill = Deno.run({
-    cmd: ["deno", "run", "-A", "--unstable", "./cli.ts", ":8080"],
+    cmd: ["deno", "run", "-A", "./cli.ts", ":8080", ":8081"],
   });
   // wait dkill finishes
   await pDkill.status();
 
   // retrieve status from test pid
-  const status = await pTest.status();
+  const status1 = await pTest1.status();
+  const status2 = await pTest2.status();
 
   // close resources
-  pTest.close();
+  pTest1.close();
+  pTest2.close();
+
   pDkill.close();
 
-  assertNotEquals(status.code, 0);
+  assertNotEquals(status1.code, 0);
+  assertNotEquals(status2.code, 0);
+
 });

cli.ts

@@ -1,4 +1,3 @@
-import { yellow } from "https://deno.land/std@0.118.0/fmt/colors.ts";
 import { Checkbox, Command } from "./deps.ts";
 import { dkill } from "./mod.ts";
 import { procList } from "./src/procList.ts";
@@ -7,8 +6,8 @@ import { assertMinVersion } from "./src/utils/versions.ts";
 
 import vJson from "./version.json" assert { type: "json" };
 
-// check minimum version of deno 1.17.0
-const minVRequired = "1.17.0";
+// check minimum version of deno
+const minVRequired = "1.29.1";
 if (!assertMinVersion(Deno.version.deno, minVRequired)) {
   console.error(`Please upgrade deno. min version required: ${minVRequired}`);
   Deno.exit(1);
@@ -25,7 +24,7 @@ await new Command()
      
     You can specify multiple targets at once: 'dkill node.exe :5000 :3000 164'`,
   )
-  .arguments("<targets...>")
+  .arguments("<...targets>")
   .option("-i, --interactive", "Interactive mode", {
     standalone: true,
   })
@@ -42,7 +41,7 @@ await new Command()
     },
   )
   .action(
-    async (opts, targets) => {
+    async (opts, ...targets) => {
       if (opts.upgrade) {
         // upgrading version.
         await upgrader({
@@ -58,11 +57,6 @@ await new Command()
       const procs: string[] = [];
 
       if (opts.interactive) {
-        if (Deno.build.os === "windows") {
-          console.warn(
-            yellow("** ON windows you cannot navigate up/down. use filter ** "),
-          );
-        }
         // list processes
         const pList = await procList();
         const pickedProcesses: string[] = await Checkbox.prompt({

deno.jsonc

@@ -1,7 +1,7 @@
 {
   "tasks": {
     "test": "deno test --allow-run --allow-net",
-    "dev": "deno run --unstable --allow-run --allow-net ./cli.ts",
+    "dev": "deno run --allow-run --allow-net ./cli.ts",
     "release": "deno run -A https://deno.land/x/release_up@0.5.0/cli.ts --regex \"(?<=@)(.*)(?=\/cli)\" --github --versionFile --changelog"
   }
 }

deno.lock

@@ -1,2 +1,2 @@
-export { Command } from "https://deno.land/x/cliffy@v0.24.2/command/mod.ts";
-export { Checkbox } from "https://deno.land/x/cliffy@v0.24.2/prompt/mod.ts";
+export { Command } from "https://deno.land/x/cliffy@v0.25.6/command/mod.ts";
+export { Checkbox } from "https://deno.land/x/cliffy@v0.25.6/prompt/mod.ts";

deps.ts

@@ -1,6 +1,6 @@
 export {
   assert,
   assertNotEquals,
-} from "https://deno.land/std@0.138.0/testing/asserts.ts";
-export { delay } from "https://deno.land/std@0.138.0/async/mod.ts";
-export { serve } from "https://deno.land/std@0.138.0/http/server.ts";
+} from "https://deno.land/std@0.170.0/testing/asserts.ts";
+export { delay } from "https://deno.land/std@0.170.0/async/mod.ts";
+export { serve } from "https://deno.land/std@0.170.0/http/server.ts";

deps_test.ts

@@ -3,7 +3,8 @@
  */
 import { serve } from "../../deps_test.ts";
 
-const port = 8080;
+const rawPort : string | number = Deno.args[0] ?? 8080;
+const port = Number(rawPort)
 
 const handler = (request: Request): Response => {
   let body = "Your user-agent is:\n\n";
@@ -12,5 +13,5 @@ const handler = (request: Request): Response => {
   return new Response(body, { status: 200 });
 };
 
-console.log(`HTTP webserver running. Access it at: http://localhost:8080/`);
+console.log(`HTTP webserver running. Access it at: http://localhost:${port}/`);
 await serve(handler, { port });

src/tests/utils.ts

@@ -2,7 +2,6 @@
 // experimental used by upgrader
 // deno-fmt-ignore
 export const denoFlags = [
-  "--unstable",
   "--allow-run",
   "--allow-net"
 ];