Skip to content
This repository has been archived by the owner on Nov 6, 2021. It is now read-only.

Split Cache module into ArtifactCache, BankCache, and possibly InventoryCache #24

Open
4 tasks
rdw-software opened this issue Aug 21, 2017 · 0 comments
Open
4 tasks

Comments

@rdw-software
Copy link
Collaborator

The Cache module after its recent rework is mostly the (formerly local) artifact cache, but some code still remains that deals with the persistent bank cache. This should be separate and tested individually to make sure nothing goes wrong that wouldn't be covered by the artifact cache unit tests.

Roadmap:

  • Split Cache into ArtifactCache and BankCache
  • Write unit tests for the new BankCache API
  • Optional: Create InventoryCache and do the same for that?
  • Rethink and maybe update the remaining (original) Cache API? Some of the code is ridiculously awful as it hasn't really changed since it was first conceived.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant