search

benetherington / node-TOM-discord-bot

MIT License
2 stars 0 forks source link

Refactor sqlite #16

Closed benetherington closed 2 years ago

benetherington commented 2 years ago

@fonji, can you take a look at this before I do any more work? There's some optimization left to do at the top of the file, but I'm mostly worried about the actual DB interfaces that get exported. Everything looks about as clean as it can get to me, but I'd like your opinion.

benetherington commented 2 years ago

Sorry to push commits into an active review! I promise I'm walking away for a bit now. :)

fonji commented 2 years ago

Don't ever be sorry about pushing commits. Just because I put myself as a reviewer does not mean I'm doing it right now anyway :)