-
Notifications
You must be signed in to change notification settings - Fork 333
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
Re: Allow for the disk cache to be stored in a different location #82
Conversation
Let's apply the last changes
…ay. Make file manager be constant to avoid potential problem with concurrency because of lazy initialization. Simplify syntax of creation
# Conflicts: # Source/Shared/BasicHybridCache.swift
@letko-dmitry, thanks for your PR! By analyzing the history of the files in this pull request, we identified @vadymmarkov and @RamonGilabert to be potential reviewers. |
Is smth wrong with it or just not needed? |
Hi @letko-dmitry. Thanks for your contribution! It looks good for me. Could you please resolve conflicts and we can merge it then. What do you guys think @zenangst @onmyway133 ? |
I think this looks good! ⭐ |
# Conflicts: # Tests/iOS/Specs/Storage/DiskStorageSpec.swift
Here it is. |
@letko-dmitry Thanks ;) |
Cool. Thank you ) |
Hi,
Let's try to complete work done in the #70. I've added simple tests to check if custom directory isn't specified by default. Are there any advices about additional tests?
Thanks in advance.