-
Notifications
You must be signed in to change notification settings - Fork 22
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
refactor caches #50
refactor caches #50
Conversation
1 similar comment
The only weird thing in this pull request is that I've moved the "cache" argument into **kwargs and don't specify it explicitly. The reason for that is if we use cache=DictCache() then I think the DictCache() gets instantiated at module load time which means that the tests don't start with a fresh DictCache object each time, thus they have previously cached responses. |
def invalidate(self, key): | ||
return self._mc.delete(self._hash(key)) | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove unneeded newlines
def invalidate(self, key): | ||
return self._mc.delete(self._hash(key)) | ||
|
||
|
||
class APIConnection(object): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 blank lines before class definition
6785dbc
to
a9a37e7
Compare
a9a37e7
to
b54ec0d
Compare
I also got rid of some pep issues in the cache.py that I found after running pep8 |
move caches into pycrest.cache module
remove cache_dir argument
update documentation about caches