knex / knex-schema-inspector

Utility for extracting information about existing DB schema
MIT License
99 stars 43 forks source link

Add support for better-sqlite3 #146

Closed faulpeltz closed 1 year ago

faulpeltz commented 1 year ago

This adds support for the better3-sqlite driver which has been supported by Knex for while now

titanism commented 1 year ago

Related https://github.com/knex/knex-schema-inspector/issues/147

titanism commented 1 year ago

@rijkvanzanten @aidenfoxx can you please merge and release this to npm? thank you 🙏

rijkvanzanten commented 1 year ago

Sure thing! FWIW I'm not sure if this library is maintained still. I've been out of this project for a while, and am not a member of the knex org (so can't archive this repo). The SQL queries don't really change necessarily, but just be aware that it might be worth running this off of a fork 👍🏻

titanism commented 1 year ago

@rijkvanzanten thank you - just ping us back once released to npm. also - you might want to regenerate the lockfile from scratch because it looks like @faulpeltz has updated it in the commit (which should be done by maintainer)

titanism commented 1 year ago

we use np for releases fyi it's great and simple https://github.com/sindresorhus/np

rijkvanzanten commented 1 year ago

we use np for releases fyi it's great and simple https://github.com/sindresorhus/np

I usually rely on a GH action for releases 🙂

rijkvanzanten commented 1 year ago

Should be up as 3.1.0 👍🏻

titanism commented 1 year ago

you're the best @rijkvanzanten 🙏 thank you

we're looking at using this for our project https://forwardemail.net - much appreciated work!

rijkvanzanten commented 1 year ago

Cool!! It's what's currently powering Directus although I migrated over to an in-house @directus/schema fork instead due to our desired move to ESM which wasn't shared by this org