feat(bentocache): Adding Drizzle ORM Adapters and Related Tests#60
feat(bentocache): Adding Drizzle ORM Adapters and Related Tests#60moor-zhou wants to merge 2 commits intoJulien-R44:mainfrom
Conversation
|
|
Thanks a lot! I will need a bit of time to review it and see if we can improve things |
Thanks for the kind words! I’m glad you like the feature. Take your time with the review—I’d be happy to address any feedback or suggestions you might have to make it even better. Looking forward to hearing your thoughts! |
|
Alright, took some time to go through the code and I'll be honest, it's a bit overwhelming. Not your fault, I know it's pretty tricky to get clean Drizzle integrations. I had looked at other packages ( like NextAuth, or Lucia If i remember well ) that provided Drizzle integrations and honestly felt the same way. That’s actually why I didn’t include Drizzle from the start (on top of the fact that I don’t use it myself). So yeah, sorry about that but I’m gonna need a bit more time : I’d like to do some refactor to try and make things a bit cleaner so it will be easier for me to maintain long-term |
|
I completely understand! Drizzle can be incredibly tricky. Your plan to refactor is smart. If you hit any snags, let’s troubleshoot together. |
No description provided.