Closed tiewweijian closed 2 years ago
@tiewweijian will do. get back to you once im done! prob by late afternoon
Merging #378 (78c991f) into master (7c89d70) will increase coverage by
0.22%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #378 +/- ##
============================================
+ Coverage 70.52% 70.75% +0.22%
+ Complexity 857 849 -8
============================================
Files 124 122 -2
Lines 2636 2602 -34
Branches 362 359 -3
============================================
- Hits 1859 1841 -18
+ Misses 670 656 -14
+ Partials 107 105 -2
Impacted Files | Coverage Δ | |
---|---|---|
...in/java/seedu/address/logic/parser/ParserUtil.java | 84.84% <ø> (-2.17%) |
:arrow_down: |
.../java/seedu/address/model/util/SampleDataUtil.java | 41.66% <ø> (+8.33%) |
:arrow_up: |
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 7c89d70...78c991f. Read the comment docs.
All instances of
tags
and their test cases has been removed ! @domlimm could you pull this PR and do some testing before merging this in? I tested on my end already but just wanted to be sure !closes #379