-
Notifications
You must be signed in to change notification settings - Fork 152
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Cache factory #430
Add Cache factory #430
Conversation
1db6b7f
to
61ad59c
Compare
Codecov Report
@@ Coverage Diff @@
## master #430 +/- ##
==========================================
+ Coverage 99.77% 99.77% +<.01%
==========================================
Files 9 10 +1
Lines 873 894 +21
Branches 96 96
==========================================
+ Hits 871 892 +21
Misses 2 2
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #430 +/- ##
==========================================
+ Coverage 99.77% 99.77% +<.01%
==========================================
Files 9 9
Lines 873 882 +9
Branches 96 96
==========================================
+ Hits 871 880 +9
Misses 2 2
Continue to review full report at Codecov.
|
@tomchristie the idea after this is to add a |
Yup good stuff! I guess ultimately I'd prefer 👍 |
0414495
to
fb181e6
Compare
Adds
aiocache.Cache
as a factory for creating different caches. Previously you would have to import the specific backend asnow
it just adds a level of indirection but will help on adding/centralizing features like creating a cache from a resource url, creating from a string for the config, etc. Also recommending this approach to create caches from now on