cache local instance #572

Merged
Plume_migration_agent merged 4 commits from cache-local-instance into master 5 years ago
Owner

fix #564

fix #564
codecov[bot] commented 5 years ago (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 5 years ago
elegaanz (Migrated from github.com) left a comment

Nice! Thank you.

Nice! Thank you.
elegaanz (Migrated from github.com) reviewed 5 years ago
elegaanz (Migrated from github.com) commented 5 years ago

Do you really need this let here?

Do you really need this `let` here?
trinity-1686a reviewed 5 years ago
Poster
Owner

just to explicitly ignore a warning about unused result

just to explicitly ignore a warning about unused result

Reviewers

The pull request has been merged as 773fbfe7c8.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b cache-local-instance master
git pull origin cache-local-instance

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff cache-local-instance
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#572
Loading…
There is no content yet.