Skip to content

Commit 497de4d

Browse files
committed
feat(fmt): align successful format write output
1 parent 274662a commit 497de4d

2 files changed

Lines changed: 19 additions & 9 deletions

File tree

‎packages/rstack/src/fmt/cli.ts‎

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -212,12 +212,22 @@ const logFmtResult = (
212212
return;
213213
}
214214

215+
if (result.exitCode === 0) {
216+
const files = `${processedFileCount} ${processedFileCount === 1 ? 'file' : 'files'}`;
217+
const details =
218+
writtenCount > 0
219+
? `${files}, ${writtenCount} formatted`
220+
: `${files}, no changes`;
221+
logger.success(
222+
`Formatting completed in ${time} ${color.dim(`(${details})`)}`,
223+
);
224+
return;
225+
}
226+
215227
const processedFiles = formatFileCount(processedFileCount);
216-
const message =
217-
writtenCount > 0
218-
? `Formatted ${formatCount(writtenCount)} of ${processedFiles} in ${time}.`
219-
: `Checked ${processedFiles} in ${time}. No changes needed.`;
220-
logger[result.exitCode === 0 ? 'success' : 'info'](message);
228+
logger.info(
229+
`Formatted ${formatCount(writtenCount)} of ${processedFiles} in ${time}.`,
230+
);
221231
return;
222232
}
223233

‎packages/rstack/tests/cli/fmt/helpers.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,11 +53,11 @@ export const expectWriteSummary = (
5353
writtenCount: number,
5454
): void => {
5555
const files = matchedFileCount === 1 ? 'file' : 'files';
56-
const message = writtenCount
57-
? `Formatted ${writtenCount} of ${matchedFileCount} ${files} in <duration>.`
58-
: `Checked ${matchedFileCount} ${files} in <duration>. No changes needed.`;
56+
const details = writtenCount
57+
? `${matchedFileCount} ${files}, ${writtenCount} formatted`
58+
: `${matchedFileCount} ${files}, no changes`;
5959
expect(normalizeDuration(output)).toBe(
60-
`start Formatting...\nsuccess ${message}\n`,
60+
`start Formatting...\nsuccess Formatting completed in <duration> (${details})\n`,
6161
);
6262
};
6363

0 commit comments

Comments
 (0)