Merge pull request #14 from ProperDocs/livereld

Fix livereload not being enabled by default with click>8.2.1

Author: oprypin

properdocs/__main__.py

@@ -254,8 +254,8 @@ def cli():
 @cli.command(name="serve")
 @click.option('-a', '--dev-addr', help=dev_addr_help, metavar='<IP:PORT>')
 @click.option('-o', '--open', 'open_in_browser', help=serve_open_help, is_flag=True)
-@click.option('--no-livereload', 'livereload', flag_value=False, help=no_reload_help)
-@click.option('--livereload', 'livereload', flag_value=True, default=True, hidden=True)
+@click.option('--no-livereload', is_flag=True, help=no_reload_help)
+@click.option('--livereload', is_flag=True, hidden=True)
 @click.option('--dirtyreload', 'build_type', flag_value='dirty', hidden=True)
 @click.option('--dirty', 'build_type', flag_value='dirty', help=serve_dirty_help)
 @click.option('-c', '--clean', 'build_type', flag_value='clean', help=serve_clean_help)
@@ -269,6 +269,11 @@ def serve_command(**kwargs):
     """Run the builtin development server."""
     from properdocs.commands import serve
 
+    if kwargs.pop('no_livereload', False):
+        kwargs['livereload'] = False
+    else:
+        kwargs['livereload'] = True
+
     _enable_warnings()
     serve.serve(**kwargs)
 

properdocs/tests/build_tests.py

@@ -586,7 +586,8 @@ def test_draft_docs_with_comments_from_user_guide(self, site_dir, docs_dir):
             docs_dir=docs_dir,
             site_dir=site_dir,
             use_directory_urls=False,
-            draft_docs='''
+            draft_docs=textwrap.dedent(
+                '''
                 # A "drafts" directory anywhere.
                 drafts/
 
@@ -595,7 +596,8 @@ def test_draft_docs_with_comments_from_user_guide(self, site_dir, docs_dir):
 
                 # But keep this particular file.
                 !/foo_unpublished.md
-            ''',
+            '''
+            ),
         )
 
         with self.subTest(serve_url=None):

properdocs/tests/config/config_options_tests.py

@@ -1135,7 +1135,7 @@ class Schema(Config):
                 r"Unrecognised theme name: 'mkdocs2'. The available installed themes are: .+"
             )
         ):
-            self.get_config(Schema, {'theme': "mkdocs2", 'plugins': "search"})
+            self.get_config(Schema, {'theme': "mkdocs2", 'plugins': ["search"]})
 
     def test_theme_default(self) -> None:
         class Schema(Config):