Commit d194d20
cli: add alias for report-directory to make it consistent
Currently every other cli option uses "dir" to reference a directory
so add an alias to allow for conistency.
PR-URL: #33587
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>1 parent c184929 commit d194d20
3 files changed
Lines changed: 6 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -635,7 +635,7 @@ Write reports in a compact format, single-line JSON, more easily consumable | |||
| 635 | 635 | by log processing systems than the default multi-line format designed for | |
| 636 | 636 | human consumption. | |
| 637 | 637 | ||
| 638 | - ### `--report-directory=directory` | ||
| 638 | + ### --report-dir=directory`, `report-directory=directory` | ||
| 639 | 639 | <!-- YAML | |
| 640 | 640 | added: v11.8.0 | |
| 641 | 641 | changes: | |
@@ -1224,7 +1224,7 @@ Node.js options that are allowed are: | |||
| 1224 | 1224 | * `--prof-process` | |
| 1225 | 1225 | * `--redirect-warnings` | |
| 1226 | 1226 | * `--report-compact` | |
| 1227 | - * `--report-directory` | ||
| 1227 | + * `--report-dir`, `--report-directory` | ||
| 1228 | 1228 | * `--report-filename` | |
| 1229 | 1229 | * `--report-on-fatalerror` | |
| 1230 | 1230 | * `--report-on-signal` | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -294,7 +294,7 @@ Write | |||
| 294 | 294 | .Sy diagnostic reports | |
| 295 | 295 | in a compact format, single-line JSON. | |
| 296 | 296 | . | |
| 297 | - .It Fl -report-directory | ||
| 297 | + .It Fl -report-dir Fl -report-directory | ||
| 298 | 298 | Location at which the | |
| 299 | 299 | .Sy diagnostic report | |
| 300 | 300 | will be generated. | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -664,11 +664,12 @@ PerProcessOptionsParser::PerProcessOptionsParser( | |||
| 664 | 664 | "output compact single-line JSON", | |
| 665 | 665 | &PerProcessOptions::report_compact, | |
| 666 | 666 | kAllowedInEnvironment); | |
| 667 | - AddOption("--report-directory", | ||
| 667 | + AddOption("--report-dir", | ||
| 668 | 668 | "define custom report pathname." | |
| 669 | - " (default: current working directory of Node.js process)", | ||
| 669 | + " (default: current working directory)", | ||
| 670 | 670 | &PerProcessOptions::report_directory, | |
| 671 | 671 | kAllowedInEnvironment); | |
| 672 | + AddAlias("--report-directory", "--report-dir"); | ||
| 672 | 673 | AddOption("--report-filename", | |
| 673 | 674 | "define custom report file name." | |
| 674 | 675 | " (default: YYYYMMDD.HHMMSS.PID.SEQUENCE#.txt)", | |
0 commit comments