From d194d208288aba559ce4c73958ff0d7801cb989c Mon Sep 17 00:00:00 2001
From: AshCripps <Ashley.Cripps@ibm.com>
Date: Wed, 27 May 2020 16:05:17 +0100
Subject: [PATCH] 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: https://github.com/nodejs/node/pull/33587
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
---
 doc/api/cli.md      | 4 ++--
 doc/node.1          | 2 +-
 src/node_options.cc | 5 +++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index 283d5b96df26f7..50ae4d5707a8cd 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -635,7 +635,7 @@ Write reports in a compact format, single-line JSON, more easily consumable
 by log processing systems than the default multi-line format designed for
 human consumption.
 
-### `--report-directory=directory`
+### --report-dir=directory`, `report-directory=directory`
 <!-- YAML
 added: v11.8.0
 changes:
@@ -1224,7 +1224,7 @@ Node.js options that are allowed are:
 * `--prof-process`
 * `--redirect-warnings`
 * `--report-compact`
-* `--report-directory`
+* `--report-dir`, `--report-directory`
 * `--report-filename`
 * `--report-on-fatalerror`
 * `--report-on-signal`
diff --git a/doc/node.1 b/doc/node.1
index 3275eadf269a4c..1a217a8bcf900a 100644
--- a/doc/node.1
+++ b/doc/node.1
@@ -294,7 +294,7 @@ Write
 .Sy diagnostic reports
 in a compact format, single-line JSON.
 .
-.It Fl -report-directory
+.It Fl -report-dir Fl -report-directory
 Location at which the
 .Sy diagnostic report
 will be generated.
diff --git a/src/node_options.cc b/src/node_options.cc
index a84420e3c5d375..cb812f6ccfc1ab 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -664,11 +664,12 @@ PerProcessOptionsParser::PerProcessOptionsParser(
             "output compact single-line JSON",
             &PerProcessOptions::report_compact,
             kAllowedInEnvironment);
-  AddOption("--report-directory",
+  AddOption("--report-dir",
             "define custom report pathname."
-            " (default: current working directory of Node.js process)",
+            " (default: current working directory)",
             &PerProcessOptions::report_directory,
             kAllowedInEnvironment);
+  AddAlias("--report-directory", "--report-dir");
   AddOption("--report-filename",
             "define custom report file name."
             " (default: YYYYMMDD.HHMMSS.PID.SEQUENCE#.txt)",