Skip to content

Commit

Permalink
tools: use built-in padStart instead of padString
Browse files Browse the repository at this point in the history
PR-URL: #17120
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
tniessen authored and MylesBorins committed Dec 11, 2017
1 parent a9be7bf commit c4eb683
Showing 1 changed file with 5 additions and 12 deletions.
17 changes: 5 additions & 12 deletions tools/lint-js.js
Original file line number Diff line number Diff line change
Expand Up @@ -215,12 +215,12 @@ if (cluster.isMaster) {

// Calculate and format the data for displaying
const elapsed = process.hrtime(startTime)[0];
const mins = padString(Math.floor(elapsed / 60), 2, '0');
const secs = padString(elapsed % 60, 2, '0');
const passed = padString(successes, 6, ' ');
const failed = padString(failures, 6, ' ');
const mins = `${Math.floor(elapsed / 60)}`.padStart(2, '0');
const secs = `${elapsed % 60}`.padStart(2, '0');
const passed = `${successes}`.padStart(6);
const failed = `${failures}`.padStart(6);
var pct = Math.ceil(((totalPaths - paths.length) / totalPaths) * 100);
pct = padString(pct, 3, ' ');
pct = `${pct}`.padStart(3);

var line = `[${mins}:${secs}|%${pct}|+${passed}|-${failed}]: ${curPath}`;

Expand All @@ -233,13 +233,6 @@ if (cluster.isMaster) {

outFn(line);
}

function padString(str, len, chr) {
str = `${str}`;
if (str.length >= len)
return str;
return chr.repeat(len - str.length) + str;
}
} else {
// Worker

Expand Down

0 comments on commit c4eb683

Please sign in to comment.