Skip to content
This repository was archived by the owner on May 20, 2025. It is now read-only.

Commit c699e60

Browse files
committed
Merge pull request #118 from Microsoft/0-percent
0.0% -> 0%
2 parents c246fad + 7e4809c commit c699e60

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

cli/script/command-executor.ts

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -776,13 +776,21 @@ function getPackageString(packageObject: Package): string {
776776

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

782782
var activePercent: number = packageObject.metrics.totalActive
783783
? packageObject.metrics.active / packageObject.metrics.totalActive * 100
784784
: 0.0;
785-
var percentString: string = (activePercent === 100.0 ? "100" : activePercent.toPrecision(2)) + "%";
785+
var percentString: string;
786+
if (activePercent === 100.0) {
787+
percentString = "100%";
788+
} else if (activePercent === 0.0) {
789+
percentString = "0%";
790+
} else {
791+
percentString = activePercent.toPrecision(2) + "%";
792+
}
793+
786794
var numPending: number = packageObject.metrics.downloaded - packageObject.metrics.installed - packageObject.metrics.failed;
787795
var returnString: string = chalk.green("Active: ") + percentString + " (" + packageObject.metrics.active.toLocaleString() + " of " + packageObject.metrics.totalActive.toLocaleString() + ")\n" +
788796
chalk.green("Total: ") + packageObject.metrics.installed.toLocaleString();

0 commit comments

Comments
 (0)