Skip to content

Commit

Permalink
Merge pull request #118 from Microsoft/0-percent
Browse files Browse the repository at this point in the history
0.0% -> 0%
  • Loading branch information
geof90 committed Jan 22, 2016
2 parents c246fad + 7e4809c commit c699e60
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions cli/script/command-executor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -776,13 +776,21 @@ function getPackageString(packageObject: Package): string {

function getPackageMetricsString(packageObject: PackageWithMetrics): string {
if (!packageObject || !packageObject.metrics) {
return "";
return "" + chalk.magenta("No installs recorded");
}

var activePercent: number = packageObject.metrics.totalActive
? packageObject.metrics.active / packageObject.metrics.totalActive * 100
: 0.0;
var percentString: string = (activePercent === 100.0 ? "100" : activePercent.toPrecision(2)) + "%";
var percentString: string;
if (activePercent === 100.0) {
percentString = "100%";
} else if (activePercent === 0.0) {
percentString = "0%";
} else {
percentString = activePercent.toPrecision(2) + "%";
}

var numPending: number = packageObject.metrics.downloaded - packageObject.metrics.installed - packageObject.metrics.failed;
var returnString: string = chalk.green("Active: ") + percentString + " (" + packageObject.metrics.active.toLocaleString() + " of " + packageObject.metrics.totalActive.toLocaleString() + ")\n" +
chalk.green("Total: ") + packageObject.metrics.installed.toLocaleString();
Expand Down

0 comments on commit c699e60

Please sign in to comment.