Codecov Report

Merging #166 into master will decrease coverage by 20.8%.
The diff coverage is 0.69%.

@@             Coverage Diff             @@
##           master     #166       +/-   ##
===========================================
- Coverage   92.64%   71.84%   -20.81%     
===========================================
  Files          92       91        -1     
  Lines        1088     1250      +162     
  Branches      165      212       +47     
===========================================
- Hits         1008      898      -110     
- Misses         65      304      +239     
- Partials       15       48       +33

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 04249d0...217c98b. Read the comment docs.

Currently WarriorJS uses 2 different approaches to generating the interfaces. Is it an idea to use Blessed for all interfaces?

@jeroenrinzema I haven't looked at the code yet, but I saw you used npm to install blessed. You should use lerna because of how the project is set up (multi-package repo using yarn workspaces). The command you need is: lerna add blessed --scope=@warriorjs/cli.

@jeroenrinzema

Currently WarriorJS uses 2 different approaches to generating the interfaces. Is it an idea to use Blessed for all interfaces?

If you mean the dialogs using Inquirer.js on one side and the printPlay() function (and all functions derived from that one) on the other, the answer is no. The scope of this PR should be limited to replace only the latter (printPlay()). All Inquirer.js dialogs should be left as they are.