[staging] Default studies_dir parameter to 'studies' and seed path to…

… seed.bin. (#1232)
brave · Oct 23, 2024 · 374e2a3 · 374e2a3
1 parent f714541
commit 374e2a3
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/src/scripts/lint.ts b/src/scripts/lint.ts
@@ -41,7 +41,7 @@ function getLintAllCommands(options: Options): string[] {
   return [
     'prettier . --ignore-unknown' + (options.fix ? ' --write' : ' --check'),
     'eslint . --config src/.eslintrc.js' + (options.fix ? ' --fix' : ''),
-    'npm run seed_tools -- lint studies' + (options.fix ? ' --fix' : ''),
+    'npm run -- seed_tools lint' + (options.fix ? ' --fix' : ''),
   ];
 }
 
@@ -52,7 +52,7 @@ function getLintDiffCommands(options: Options): Record<string, any> {
     '*.{ts,js,tsx,jsx}':
       'eslint --config src/.eslintrc.js' + (options.fix ? ' --fix' : ''),
     'studies/*': () =>
-      'npm run seed_tools -- lint studies' + (options.fix ? ' --fix' : ''),
+      'npm run -- seed_tools lint' + (options.fix ? ' --fix' : ''),
   };
 }
 

diff --git a/src/seed_tools/commands/create.ts b/src/seed_tools/commands/create.ts
@@ -12,8 +12,12 @@ import { readStudiesToSeed } from '../utils/studies_to_seed';
 export default function createCommand() {
   return new Command('create')
     .description('Create seed.bin from study files')
-    .argument('<studies_dir>', 'path to the directory containing study files')
-    .argument('<output_seed_file>', 'output seed file')
+    .argument(
+      '[studies_dir]',
+      'path to the directory containing study files',
+      'studies',
+    )
+    .argument('[output_seed_file]', 'output seed file', 'seed.bin')
     .option('--mock_serial_number <value>', 'mock serial number')
     .option(
       '--output_serial_number_file <path>',

diff --git a/src/seed_tools/commands/lint.ts b/src/seed_tools/commands/lint.ts
@@ -9,7 +9,11 @@ import { readStudiesToSeed } from '../utils/studies_to_seed';
 export default function createCommand() {
   return new Command('lint')
     .description('Lint study files without creating seed.bin')
-    .argument('<studies_dir>', 'path to the directory containing study files')
+    .argument(
+      '[studies_dir]',
+      'path to the directory containing study files',
+      'studies',
+    )
     .option('--fix', 'fix format errors in-place')
     .action(lintStudies);
 }