Skip to content

Commit 11fdf66

Browse files
committed
refactor(fmt): centralize worker option parsing
1 parent 870396b commit 11fdf66

1 file changed

Lines changed: 12 additions & 11 deletions

File tree

packages/rstack/src/fmt/cli.ts

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,17 @@ ${color.cyan('Options')}:
3030
--parallel-workers <count> Number of parallel workers
3131
-h, --help Display this help message`;
3232

33-
const parseMaxWorkers = (value: string | undefined): number | undefined => {
33+
const parseMaxWorkers = (
34+
kebabValue: string | undefined,
35+
camelValue: string | undefined,
36+
): number | undefined => {
37+
if (kebabValue !== undefined && camelValue !== undefined) {
38+
throw new Error(
39+
'The --parallel-workers and --parallelWorkers options cannot be used together.',
40+
);
41+
}
42+
43+
const value = kebabValue ?? camelValue;
3444
if (value === undefined) {
3545
return undefined;
3646
}
@@ -69,16 +79,7 @@ const parseFmtCLIArgs = (args: string[]): ParsedFmtCLIArgs => {
6979

7080
const mode = values.check ? 'check' : listDifferent ? 'list-different' : 'write';
7181
const noParallel = values['no-parallel'] || values.noParallel;
72-
const kebabMaxWorkers = values['parallel-workers'];
73-
const camelMaxWorkers = values.parallelWorkers;
74-
75-
if (kebabMaxWorkers !== undefined && camelMaxWorkers !== undefined) {
76-
throw new Error(
77-
'The --parallel-workers and --parallelWorkers options cannot be used together.',
78-
);
79-
}
80-
81-
const maxWorkers = parseMaxWorkers(kebabMaxWorkers ?? camelMaxWorkers);
82+
const maxWorkers = parseMaxWorkers(values['parallel-workers'], values.parallelWorkers);
8283

8384
if (noParallel && maxWorkers !== undefined) {
8485
throw new Error('The --parallel-workers and --no-parallel options cannot be used together.');

0 commit comments

Comments
 (0)