cache local instance #572

Merged

Plume_migration_agent merged 4 commits from cache-local-instance into master

2019-05-10 20:59:34 +00:00

trinity-1686a commented

2019-05-10 14:01:10 +00:00

Owner

fix #564

codecov[bot] commented

2019-05-10 14:14:43 +00:00

(Migrated from github.com)

Codecov Report

Merging #572 into master will decrease coverage by 0.2%.
The diff coverage is 36.73%.

@@            Coverage Diff             @@
##           master     #572      +/-   ##
==========================================
- Coverage   34.72%   34.51%   -0.21%     
==========================================
  Files          67       67              
  Lines        7851     7872      +21     
  Branches     1886     1890       +4     
==========================================
- Hits         2726     2717       -9     
- Misses       4367     4394      +27     
- Partials      758      761       +3

# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/572?src=pr&el=h1) Report > Merging [#572](https://codecov.io/gh/Plume-org/Plume/pull/572?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/90f6ef0380fb3209a3c68963ba32cf85bdd72112?src=pr&el=desc) will **decrease** coverage by `0.2%`. > The diff coverage is `36.73%`. ```diff @@ Coverage Diff @@ ## master #572 +/- ## ========================================== - Coverage 34.72% 34.51% -0.21% ========================================== Files 67 67 Lines 7851 7872 +21 Branches 1886 1890 +4 ========================================== - Hits 2726 2717 -9 - Misses 4367 4394 +27 - Partials 758 761 +3 ```

elegaanz (Migrated from github.com) approved these changes 2019-05-10 18:34:59 +00:00

elegaanz (Migrated from github.com) left a comment

Nice! Thank you.

elegaanz (Migrated from github.com) reviewed 2019-05-10 18:35:45 +00:00

plume-cli/src/main.rs

elegaanz (Migrated from github.com) commented

2019-05-10 18:35:45 +00:00

Do you really need this let here?

Do you really need this `let` here?