Merge pull request #23 from Enzojz/main

Fix to runInContext, runInNewContext and runInThisContext signature
TheSpyder · Jan 24, 2024 · 0e1f74e · 0e1f74e
2 parents 4cc5c5a + 8965726
commit 0e1f74e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.changes/unreleased/Fixed-20240124-121124.yaml b/.changes/unreleased/Fixed-20240124-121124.yaml
@@ -0,0 +1,5 @@
+kind: Fixed
+body: Corrected `VM` method signatures `runInContext`, `runInNewContext` and `runInThisContext`
+time: 2024-01-24T12:11:24.898655+11:00
+custom:
+  GithubIssue: "23"
diff --git a/src/VM.res b/src/VM.res
@@ -25,8 +25,8 @@ module Script = {
   external makeWithOptions: (string, options) => t = "Script"
   @send external createCachedData: t => Buffer.t = "createCachedData"
   @send
-  external runInContext: (t, string, contextifiedObject<'a>) => 'b = "runInContext"
+  external runInContext: (t, contextifiedObject<'a>) => 'b = "runInContext"
   @send
-  external runInNewContext: (t, string, contextifiedObject<'a>) => 'b = "runInNewContext"
-  @send external runInThisContext: (t, string) => 'a = "runInThisContext"
+  external runInNewContext: (t,  contextifiedObject<'a>) => 'b = "runInNewContext"
+  @send external runInThisContext: t => 'a = "runInThisContext"
 }