Closed glneto closed 6 years ago
Merging #121 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #121 +/- ##
=======================================
Coverage 90.87% 90.87%
=======================================
Files 80 80
Lines 1019 1019
Branches 158 158
=======================================
Hits 926 926
Misses 77 77
Partials 16 16
Impacted Files | Coverage Δ | |
---|---|---|
...ackages/warriorjs-cli/src/ui/printWarriorStatus.js | 100% <ø> (ø) |
: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 1872e0c...530e2a8. Read the comment docs.
@olistic Alright! I didn't see the is-windows package wasn't being used. Should be good now :)
Hello!
This change should fix the diamond symbol on Windows' CMD. I used the is-windows util to check the user env and in case that returns true, I use the Black Diamond Suit (as suggested on the issue), otherwise I keep the black medium diamond symbol.
I hope that helps.
Fixes #112