Merge pull request #928 from thefrontside/deno-2

.github/workflows/npm-release.yml

@@ -17,18 +17,17 @@ jobs:
 
     steps:
       - name: checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: setup deno
-        uses: denoland/setup-deno@v1
+        uses: denoland/setup-deno@v2
         with:
-          deno-version: v1.x
+          deno-version: v2.x
 
       - name: Get Version
         id: vars
         run: echo ::set-output name=version::$(echo ${{github.ref_name}} | sed 's/^effection-v//')
 
-
       - name: Setup Node
         uses: actions/setup-node@v2
         with:

.github/workflows/verify.yaml

@@ -15,12 +15,12 @@ jobs:
 
     steps:
       - name: checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: setup deno
-        uses: denoland/setup-deno@v1
+        uses: denoland/setup-deno@v2
         with:
-          deno-version: v1.46.x
+          deno-version: v2.x
 
       - name: Setup Node
         uses: actions/setup-node@v2

lib/call.ts

@@ -123,7 +123,7 @@ export function call<T>(callable: Callable<T>): Operation<T> {
         resolve(yield* toop(callable));
       }
     } catch (error) {
-      reject(error);
+      reject(error as Error);
     }
   });
 }

lib/context.ts

@@ -27,5 +27,5 @@ export function createContext<T>(key: string, defaultValue?: T): Context<T> {
 }
 
 class MissingContextError extends Error {
-  name = "MissingContextError";
+  override name = "MissingContextError";
 }

lib/main.ts

@@ -113,7 +113,7 @@ export async function main(
 
         yield* exit(0);
       } catch (error) {
-        resolve({ status: 1, error });
+        resolve({ status: 1, error: error as Error });
       } finally {
         clearInterval(interval);
       }

lib/race.ts

@@ -33,7 +33,7 @@ export function race<T extends Operation<unknown>>(
         try {
           resolve((yield* operation) as Yielded<T>);
         } catch (error) {
-          reject(error);
+          reject(error as Error);
         }
       });
     }

lib/run/frame.ts

@@ -98,7 +98,7 @@ export function createFrame<T>(options: FrameOptions<T>): Frame<T> {
                 result: yield* instruction(frame),
               });
             } catch (error) {
-              k.tail({ type: "settled", result: Err(error) });
+              k.tail({ type: "settled", result: Err(error as Error) });
             }
           });
 
@@ -117,7 +117,7 @@ export function createFrame<T>(options: FrameOptions<T>): Frame<T> {
           }
         }
       } catch (error) {
-        thunks.unshift({ done: true, value: Err(error) });
+        thunks.unshift({ done: true, value: Err(error as Error) });
       }
       thunk = thunks.pop()!;
     }

test/queue.test.ts

@@ -58,7 +58,7 @@ function abortAfter<T>(op: Operation<T>, ms: number): Operation<T | void> {
       try {
         resolve(yield* op);
       } catch (error) {
-        reject(error);
+        reject(error as Error);
       }
     });
     yield* sleep(ms);

test/run.test.ts

@@ -257,7 +257,7 @@ describe("run()", () => {
       });
       throw new Error("expected error to propagate");
     } catch (error) {
-      expect(error.message).toEqual("boom");
+      expect((error as Error).message).toEqual("boom");
     }
   });
 
@@ -268,7 +268,7 @@ describe("run()", () => {
       });
       throw new Error("expected error to propagate");
     } catch (error) {
-      expect(error.message).toEqual("boom");
+      expect((error as Error).message).toEqual("boom");
     }
   });
 

test/suite.ts

@@ -96,7 +96,9 @@ export function useCommand(
 
         if (
           !!error &&
-          !error.message.includes("Child process has already terminated")
+          !(error as Error).message.includes(
+            "Child process has already terminated",
+          )
         ) {
           // deno-lint-ignore no-unsafe-finally
           throw error;