Repository: nteract/papermill

Issue:
Typo in anti-flag for `--report-mode`
There's a simple typo in the definition of `--report-mode`:

https://github.com/nteract/papermill/blob/01ec212f14c76860f3788f6f92f9be27d5742ea3/papermill/cli.py#L150

All other flag values have an "anti-flag" of `--no-something`, `--report-mode` has `--not-report-mode` (_not_ vs _no_).


Discussion:

Patch:
diff --git a/docs/reference/papermill-cli.rst b/docs/reference/papermill-cli.rst
index 14cd680..426e266 100644
--- a/docs/reference/papermill-cli.rst
+++ b/docs/reference/papermill-cli.rst
@@ -64,7 +64,7 @@ Command Line options
       --log-level [NOTSET|DEBUG|INFO|WARNING|ERROR|CRITICAL]
                                       Set log level
       --start_timeout INTEGER         Time in seconds to wait for kernel to start.
-      --report-mode / --not-report-mode
+      --report-mode / --no-report-mode
                                       Flag for hiding input.
       --version                       Flag for displaying the version.
       -h, --help                      Show this message and exit.
diff --git a/docs/usage-cli.rst b/docs/usage-cli.rst
index 14e8424..b25399b 100644
--- a/docs/usage-cli.rst
+++ b/docs/usage-cli.rst
@@ -55,7 +55,7 @@ options:
       --log-level [NOTSET|DEBUG|INFO|WARNING|ERROR|CRITICAL]
                                       Set log level
       --start_timeout INTEGER         Time in seconds to wait for kernel to start.
-      --report-mode / --not-report-mode
+      --report-mode / --no-report-mode
                                       Flag for hiding input.
       --version                       Flag for displaying the version.
       -h, --help                      Show this message and exit.
diff --git a/papermill/cli.py b/papermill/cli.py
index a2e5eb4..3ba32d7 100755
--- a/papermill/cli.py
+++ b/papermill/cli.py
@@ -157,7 +157,7 @@ class OptionEatAll(click.Option):
 @click.option(
     '--start_timeout', type=int, default=60, help="Time in seconds to wait for kernel to start."
 )
-@click.option('--report-mode/--not-report-mode', default=False, help="Flag for hiding input.")
+@click.option('--report-mode/--no-report-mode', default=False, help="Flag for hiding input.")
 @click.option(
     '--version',
     is_flag=True,


Test patch:
diff --git a/papermill/tests/test_cli.py b/papermill/tests/test_cli.py
index ea81b26..5dc6aed 100644
--- a/papermill/tests/test_cli.py
+++ b/papermill/tests/test_cli.py
@@ -326,7 +326,7 @@ class TestCLI(unittest.TestCase):
 
     @patch(cli.__name__ + '.execute_notebook')
     def test_no_report_mode(self, execute_patch):
-        self.runner.invoke(papermill, self.default_args + ['--not-report-mode'])
+        self.runner.invoke(papermill, self.default_args + ['--no-report-mode'])
         execute_patch.assert_called_with(**self.augment_execute_kwargs(report_mode=False))
 
     @patch(cli.__name__ + '.execute_notebook')