Closed agubler closed 6 years ago
Merging #222 into master will decrease coverage by
0.02%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #222 +/- ##
==========================================
- Coverage 95.06% 95.04% -0.03%
==========================================
Files 19 19
Lines 507 525 +18
Branches 69 70 +1
==========================================
+ Hits 482 499 +17
Misses 13 13
- Partials 12 13 +1
Impacted Files | Coverage Δ | |
---|---|---|
src/registerCommands.ts | 99.02% <100%> (-0.98%) |
: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 0c13fcb...db7dbd0. Read the comment docs.
Type: bug
The following has been addressed in the PR:
prettier
as per the readme code style guidelinesDescription:
Parsed aliases are only available from yargs when running a command with a group and name - Therefore running a default command such as
dojo test
throws an error because there is no aliases object.This change moves from using an internally yargs generated and maintained aliases map, to explicitly maintaining an equivalent map in dojo/cli. This means that command line options saved in the dojorc config will work for default commands, aliased commands and regular commands.
Resolves #221