refactor: Don't require Channel.Config.t's "json" prop

amsross · Mar 17, 2020 · 19c97e3 · 19c97e3
1 parent aba40e7
commit 19c97e3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 11 deletions.
diff --git a/src/AmqpConnectionManager.re b/src/AmqpConnectionManager.re
@@ -22,11 +22,7 @@ module Channel = {
   [@bs.send] external nack: (t, Queue.message) => unit = "nack";
 
   module Config = {
-    type nonrec t = {
-      .
-      "json": bool,
-      "setup": t => Js.Promise.t(unit),
-    };
+    type nonrec t('a) = {.. "setup": t => Js.Promise.t(unit)} as 'a;
   };
 
   [@bs.send]
@@ -130,7 +126,7 @@ module AmqpConnectionManager = {
     "on";
 
   [@bs.send]
-  external createChannel: (t, Channel.Config.t) => ChannelWrapper.t =
+  external createChannel: (t, Channel.Config.t('a)) => ChannelWrapper.t =
     "createChannel";
 };
 

diff --git a/src/AmqpConnectionManager.rei b/src/AmqpConnectionManager.rei
@@ -20,11 +20,7 @@ module Channel: {
   let nack: t => nack;
 
   module Config: {
-    type nonrec t = {
-      .
-      "json": bool,
-      "setup": t => Js.Promise.t(unit),
-    };
+    type nonrec t('a) = {.. "setup": t => Js.Promise.t(unit)} as 'a;
   };
 
   let assertExchange: