Support specifying a knex instance instead of config object (#52)

* Support specifying a knex instance instead of config object

* bump version and update changelog

* npm audit fix

Co-authored-by: Charles Burgess <[email protected]>

Author: theogravity

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v1.2.0 - 04-07-2020
+
+- Support specifying a knex instance instead of config object ( Thanks @theogravity! )
+
 ## v1.1.1 - 01-23-2020
 
 - Knex update for security

README.md

@@ -58,6 +58,7 @@ const knexConfig = {
   }
 };
 
+// you can also pass a knex instance instead of a configuration object
 const db = new MyDatabase(knexConfig);
 
 const server = new ApolloServer({

index.js

@@ -14,7 +14,13 @@ class SQLDataSource extends DataSource {
 
     this.context;
     this.cache;
-    this.db = Knex(knexConfig);
+
+    if (typeof knexConfig === "function") {
+      this.db = knexConfig;
+    } else {
+      this.db = Knex(knexConfig);
+    }
+
     this.knex = this.db;
 
     const _this = this;