kellegous / go

Another Google-like Go short link service
MIT License
584 stars 122 forks source link

Encapsulate backends #22

Closed adamallred closed 4 years ago

adamallred commented 4 years ago

This PR:

I also tested this by installing the current version of go and running it over a pre-existing leveldb that was made with this encapsulation, and confirmed that functionality looked good. It seems to work both forwards and backwards

It's kinda large. I can probably split it by moving to the vendor module in a separate PR, and also by adding passing of context.Context separately as well. Let me know if you'd like it split up.