Baseflow / flutter_cache_manager

Generic cache manager for flutter
https://baseflow.com
MIT License
749 stars 438 forks source link

Added option to migrate between repos #230

Closed renefloor closed 4 years ago

renefloor commented 4 years ago

:sparkles: What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Feature

:arrow_heading_down: What is the current behavior?

There is no easy way to migrate to newer CacheInfoRepositories

:new: What is the new behavior (if this is a feature change)?

:boom: Does this PR introduce a breaking change?

Yes, other implementations of CacheInfoRepository will need to change.

:bug: Recommendations for testing

:memo: Links to relevant issues/docs

:thinking: Checklist before submitting

codecov[bot] commented 4 years ago

Codecov Report

Merging #230 into develop will decrease coverage by 0.39%. The diff coverage is 56.25%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #230      +/-   ##
===========================================
- Coverage    75.31%   74.91%   -0.40%     
===========================================
  Files           17       19       +2     
  Lines          559      618      +59     
===========================================
+ Hits           421      463      +42     
- Misses         138      155      +17     
Impacted Files Coverage Δ
...cache_info_repositories/cache_object_provider.dart 0.00% <0.00%> (ø)
...info_repositories/non_storing_object_provider.dart 0.00% <0.00%> (ø)
..._info_repositories/json_cache_info_repository.dart 92.78% <79.41%> (+1.32%) :arrow_up:
...cache_info_repositories/cache_info_repository.dart 94.11% <94.11%> (ø)
...torage/cache_info_repositories/helper_methods.dart 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9339159...39fdd59. Read the comment docs.