Forbes Lindesay

Results 256 comments of Forbes Lindesay

If anyone can provide a sample of how to do this using the underlying `mysql2` library, I'd be happy to take a look at this. Better yet, someone could provide...

I think this is specific to SQLite, because we use [`Database.all`](https://github.com/mapbox/node-sqlite3/wiki/API#databaseallsql-param--callback) to run the underlying query and that API only supports passing a single query. Overall this is pretty inconsistent...

`then-queue` is only used if you use `queryStream` so that isn't the culprit here. It could be caused by the fact that we're using babel to target node 12, so...

This needs updating in the documentation to show using an array. Using an array is much more reliable than using a string would be, so it makes sense to prefer...

Yes, we already support something very similar for Postgres. I'm happy to accept a pull request that allows specifying the connection configuration as an object.

I'd be happy to accept a pull request for this.

I think this should be achievable by using a "Keyword/Value Connection String" (see https://www.postgresql.org/docs/10/libpq-connect.html) This would look something like: ```sh npx '@databases/pg-schema-cli' \ --database "host='/var/run/postgresql/' user='$POSTGRES_USER' password='$POSTGRES_PASSWORD' dbname='$POSTGRES_DB'" \ --directory...

Could you share a sample of the code used to change database when using the default mysql2 driver?

I think the only option where would be to use `.mjs` for your modules, instead of relying on `"type": "module"` in `package.json`. I don't think there's any obvious way to...

I would be happy to accept a PR for this. This can be added as a config option in https://github.com/ForbesLindesay/atdatabases/blob/8f7f7a91efc7d3095f242d5c1d361b4458a92e60/packages/pg-config/src/PgConfig.ts#L133