Make closeDB return an Aff instead an Effect, like the other functions.

[Dretch]

Can closeDB return an Aff?

This would mean API users have to do less converting between Aff and Effect

[Dretch]

Hmmm, my proposed change is not safe: the cb function needs to be called, otherwise the closeDB call will never complete.

The original closeDB using Effect is also not safe, though, since it won't wait for the close to actually complete.

I'll change this.

[Dretch]

I think this is good now. Please consider this change!

[Dretch]

Bump @justinwoo 🙏

[justinwoo]

Oh, sorry. Looks good, maybe eventually could use a closeDB' or something for the Effect version.

[justinwoo]

Made https://github.com/justinwoo/purescript-node-sqlite3/releases/tag/v5.0.0