quick PR self review

Author: Archmonger

src/reactpy_django/components.py

@@ -319,8 +319,8 @@ def _python_to_pyscript(
     executor = render_pyscript_template(file_path, uuid, root)
 
     if not rendered:
-        # FIXME: This is needed to properly re-render PyScript instances such as
-        # when a component is re-rendered due to WebSocket disconnection.
+        # FIXME: This is needed to properly re-render PyScript such as
+        # during a WebSocket disconnection / reconnection.
         # There may be a better way to do this in the future.
         set_rendered(True)
         return None

src/reactpy_django/management/commands/clean_reactpy.py

@@ -28,10 +28,10 @@ def add_arguments(self, parser):
         parser.add_argument(
             "--sessions",
             action="store_true",
-            help="Configure this clean to only clean session data (and other configured cleaning options).",
+            help="Clean session data. This value can be combined with other cleaning options.",
         )
         parser.add_argument(
             "--user-data",
             action="store_true",
-            help="Configure this clean to only clean user data (and other configured cleaning options).",
+            help="Clean user data. This value can be combined with other cleaning options.",
         )

src/reactpy_django/templatetags/reactpy.py

@@ -231,7 +231,6 @@ def pyscript_setup(
     *extra_packages: str,
     config: str | dict = "",
 ):
-    print(reactpy_config.REACTPY_DEBUG_MODE)
     return {
         "pyscript_config": extend_pyscript_config(config, extra_packages),
         "pyscript_layout_handler": PYSCRIPT_LAYOUT_HANDLER,

src/reactpy_django/utils.py

@@ -461,7 +461,7 @@ def vdom_or_component_to_string(
 
 
 def render_pyscript_template(file_path: str, uuid: str, root: str):
-    """Inserts the user's code into our PyScript template using pattern matching."""
+    """Inserts the user's code into the PyScript template using pattern matching."""
     # Create a valid PyScript executor by replacing the template values
     executor = PYSCRIPT_COMPONENT_TEMPLATE.replace("UUID", uuid)
     executor = executor.replace("return root()", f"return {root}()")
@@ -476,7 +476,7 @@ def render_pyscript_template(file_path: str, uuid: str, root: str):
 
 
 def extend_pyscript_config(config: dict | str, extra_packages: Sequence) -> str:
-    """Extends the default PyScript configuration with user configuration."""
+    """Extends ReactPy's default PyScript config with user provided values."""
     pyscript_config = deepcopy(PYSCRIPT_DEFAULT_CONFIG)
     pyscript_config["packages"].extend(extra_packages)
     if config and isinstance(config, str):

tests/test_app/__init__.py

@@ -8,7 +8,7 @@
 assert npm.call(["install"], cwd=str(js_dir)) == 0
 assert npm.call(["run", "build"], cwd=str(js_dir)) == 0
 
-# Make sure the current PyScript distribution is available
+# Make sure the the PyScript distribution is always available
 pyscript_dist = js_dir / "node_modules" / "@pyscript" / "core" / "dist"
 pyscript_static_dir = (
     Path(__file__).parent.parent.parent