Closed aaronrothschild closed 4 years ago
@aaronrothschild I've updated your branch and merged @jfrerich's suggestion. Hope you are fine with this.
@jfrerich PTAL
Merging #38 into master will increase coverage by
2.04%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #38 +/- ##
==========================================
+ Coverage 26.23% 28.27% +2.04%
==========================================
Files 6 6
Lines 343 343
==========================================
+ Hits 90 97 +7
+ Misses 235 229 -6
+ Partials 18 17 -1
Impacted Files | Coverage Δ | |
---|---|---|
server/command.go | 0.00% <ø> (ø) |
|
server/utils.go | 95.12% <0.00%> (+17.07%) |
: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 0f165d1...da3ad91. Read the comment docs.
These changes are very straight forward. Let's merge them without QA review.
Changed some help text info.
I haven't tested this locally.