Aight, I think I addressed your feedback @fitzoh , and tests are still passing, no lint complaints, dogfooding, and lots of other cool stuff. We reduced our dependencies quite a bit by doing this, AND it's cross platform AND it's STUPID fast at least locally
I stepped through it a few times with the debugger to make sure it was doing exactly what I wanted
Either way we go forward now, we at least have the cache more or less extracted from ossindex itself, so it should be easyish to swap to another one if pudge isn't making the cut
ANNNND cache specific tests added. Largely duplicate the OSS Index ones, but a bit more fine grained
And that said, I AM OUTTA HERE!
Hmmm there is an intermittent failure
The tests being in parallel now I'm having some fun
I rethought those packages from a "I'm a consumer" standpoint
So function names, Options, etc... all kinda rethunk
@DarthHater the first rule of channels is don't use them.
in this case, it's doesn't appear useful and should be removed (imho)
If you can rethink that long polling process, I'm more than game. I used a goroutine and channel to block the process from exiting because it was the only way I knew how to, but I surmise I just didn't really grasp something else!