fix: improve the commands output

CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [v0.1.1]
+
+### Fixed
+
+- Improve the commands output.
+
+## [v0.1.0]
+
+### Added
+
+- Picasso commands in simple way.

tutorpicasso/__about__.py

@@ -1 +1 @@
-__version__ = "0.1.0"
+__version__ = "0.1.1"

tutorpicasso/commands/enable_private_packages.py

@@ -6,6 +6,8 @@
 from packaging.version import Version
 from tutor import config as tutor_config
 from tutor.__about__ import __version__ as tutor_version
+from tutor import utils as tutor_utils
+from tutor import fmt as tutor_fmt
 
 
 @click.command(name="enable-private-packages", help="Enable picasso private packages")
@@ -35,14 +37,12 @@ def enable_private_packages() -> None:
                 f"{package} is missing one of the required keys: 'name', 'repo', 'version'"
             )
 
-        if os.path.isdir(f'{private_requirements_path}/{info["name"]}'):
-            subprocess.call(
-                ["rm", "-rf", f'{private_requirements_path}/{info["name"]}']
-            )
+        requirement_path = f'{private_requirements_path}/{info["name"]}'
+        if os.path.isdir(requirement_path):
+            tutor_utils.execute("rm", "-rf", requirement_path)
 
-        subprocess.call(
-            ["git", "clone", "-b", info["version"], info["repo"]],
-            cwd=private_requirements_path,
+        tutor_utils.execute(
+            "git", "clone", "-b", info["version"], info["repo"], requirement_path
         )
 
         handle_private_requirements_by_tutor_version(info, private_requirements_path)
@@ -84,6 +84,7 @@ def _enable_private_requirements_before_quince(
 
     echo_command = f'echo "-e ./{info["name"]}/" >> {private_requirements_txt}'
     subprocess.call(echo_command, shell=True)
+    click.echo(tutor_fmt.command(echo_command))
 
 
 def _enable_private_requirements_latest(
@@ -96,13 +97,8 @@ def _enable_private_requirements_latest(
         info (Dict[str, str]): A dictionary containing metadata about the requirement. Expected to have a "name" key.
         private_requirements_path (str): The root directory where private requirements are stored.
     """
-    subprocess.call(
-        [
-            "tutor",
-            "mounts",
-            "add",
-            f'{private_requirements_path}/{info["name"]}',
-        ]
+    tutor_utils.execute(
+        "tutor", "mounts", "add", f'{private_requirements_path}/{info["name"]}'
     )
 
 

tutorpicasso/commands/enable_themes.py

@@ -4,6 +4,7 @@
 
 import click
 from tutor import config as tutor_config
+from tutor import utils as tutor_utils
 
 
 @click.command(name="enable-themes", help="Enable picasso themes")
@@ -25,16 +26,29 @@ def enable_themes() -> None:
     # because it comes from the Tutor framework.
     # We are not handle type errors related to this object.
     for theme in tutor_conf["PICASSO_THEMES"]:  # type: ignore
-        if not {"name", "repo", "version"}.issubset(theme.keys()):  # type: ignore
+        if not isinstance(theme, dict):
             raise click.ClickException(
-                f"{theme} is missing one or more required keys: "
-                "'name', 'repo', 'version'"
+                "Expected 'theme' to be a dictionary, but got something else."
             )
 
-        theme_path = f'{tutor_root}/env/build/openedx/themes/{theme["name"]}'  # type: ignore
-        if os.path.isdir(theme_path):
-            subprocess.call(["rm", "-rf", theme_path])
-
-        subprocess.call(
-            ["git", "clone", "-b", theme["version"], theme["repo"], theme_path],  # type: ignore
-        )
+        else:
+            if not {"name", "repo", "version"}.issubset(theme.keys()):
+                raise click.ClickException(
+                    f"{theme} is missing one or more required keys: "
+                    "'name', 'repo', 'version'"
+                )
+
+            theme_path = f'{tutor_root}/env/build/openedx/themes/{theme["name"]}'
+            if os.path.isdir(theme_path):
+                subprocess.call(["rm", "-rf", theme_path])
+
+            theme_version = theme.get("version", "")
+            theme_repo = theme.get("repo", "")
+            tutor_utils.execute(
+                "git",
+                "clone",
+                "-b",
+                str(theme_version),
+                str(theme_repo),
+                str(theme_path),
+            )

tutorpicasso/commands/run_extra_commands.py

@@ -6,6 +6,7 @@
 
 import click
 from tutor import config as tutor_config
+from tutor import fmt as tutor_fmt
 
 COMMAND_CHAINING_OPERATORS = ["&&", "&", "||", "|", ";"]
 
@@ -80,6 +81,7 @@ def run_command(command: str) -> None:
     Args:
         command (str): Tutor command.
     """
+    click.echo(tutor_fmt.command(command))
     with subprocess.Popen(
         command,
         shell=True,