zold-io / java-api

Zold Java API
https://www.zold.io
MIT License
21 stars 11 forks source link

#17 - implement Wallet.File.ledger() #30

Closed t-izbassar closed 6 years ago

t-izbassar commented 6 years ago

Fix for #17. Implemented ledger() method. Updated relevant test case.

0crat commented 6 years ago

Job #30 is now in scope, role is REV

0crat commented 6 years ago

Job #30 is already in scope

0crat commented 6 years ago

This pull request #30 is assigned to @carlosmiranda/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @llorllale/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

codecov-io commented 6 years ago

Codecov Report

Merging #30 into master will increase coverage by 0.26%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #30      +/-   ##
=========================================
+ Coverage     94.73%   95%   +0.26%     
  Complexity       17    17              
=========================================
  Files             3     3              
  Lines            38    40       +2     
  Branches          1     1              
=========================================
+ Hits             36    38       +2     
  Misses            2     2
Impacted Files Coverage Δ Complexity Δ
src/main/java/io/zold/api/Wallet.java 100% <100%> (ø) 0 <0> (ø) :arrow_down:

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 1405fe3...f524275. Read the comment docs.

t-izbassar commented 6 years ago

@carlosmiranda updated.

t-izbassar commented 6 years ago

@llorllale merge?

llorllale commented 6 years ago

@rultor merge

rultor commented 6 years ago

@rultor merge

@llorllale OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 6 years ago

@rultor merge

@llorllale Done! FYI, the full log is here (took me 11min)

0crat commented 6 years ago

@ypshenychka/z please review this job completed by @carlosmiranda/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

0crat commented 6 years ago

Job gh:zold-io/java-api#30 is not assigned, can't get performer

0crat commented 6 years ago

The job #30 is now out of scope

0crat commented 6 years ago

Payment to ARC for a closed pull request, as in §28: +10 point(s) just awarded to @llorllale/z

ypshenychka commented 6 years ago

@carlosmiranda According to our QA Rules:

The code reviewer found at least three problems in the code. Comments were mostly about design problems, not cosmetic issues.

Two out of four found issues are cosmetic. Please confirm that you'll try to find at least three major problems while future reviews.

carlosmiranda commented 6 years ago

@ypshenychka confirmed

ypshenychka commented 6 years ago

@carlosmiranda thanks

ypshenychka commented 6 years ago

@0crat quality acceptable

0crat commented 6 years ago

Order was finished, quality is "acceptable": +15 point(s) just awarded to @carlosmiranda/z

0crat commented 6 years ago

Quality review completed: +8 point(s) just awarded to @ypshenychka/z

g4s8 commented 6 years ago

@0crat status

0crat commented 6 years ago

@0crat status (here)

@g4s8 This is what I know about this job in CAZUREFND, as in §32: