Add option to disable scripts

CHANGELOG.md

@@ -1,6 +1,7 @@
 # 3.4.1 (beta)
 
 - Fixed logic in filtering nested config.
+- Added `--no-scripts` option to skip install/update scripts.
 
 # 3.4 (2023-03-24)
 

gitman/cli.py

@@ -118,6 +118,13 @@ def main(args=None, function=None):
         action="store_true",
         dest="no_defaults",
     )
+    sub.add_argument(
+        "-S",
+        "--no-scripts",
+        help="skip running scripts after installation",
+        action="store_true",
+        dest="no_scripts",
+    )
 
     # Update parser
     info = "update dependencies to the latest versions"
@@ -144,6 +151,13 @@ def main(args=None, function=None):
         default=None,
         help="disable recording of updated versions",
     )
+    sub.add_argument(
+        "-S",
+        "--no-scripts",
+        help="skip running scripts after installation",
+        action="store_true",
+        dest="no_scripts",
+    )
 
     # List parser
     info = "display the current version of each dependency"
@@ -262,6 +276,7 @@ def _get_command(function, namespace):
             force_interactive=namespace.force_interactive,
             clean=namespace.clean,
             skip_changes=namespace.skip_changes,
+            skip_scripts=namespace.no_scripts,
         )
         if namespace.command == "install":
             kwargs.update(

gitman/commands.py

@@ -62,6 +62,7 @@ def install(
     clean=True,
     skip_changes=False,
     skip_default_group=False,
+    skip_scripts=False,
 ):
     """Install dependencies for a project.
 
@@ -80,6 +81,7 @@ def install(
      should be skipped
     - `skip_default_group`: indicates default_group should be skipped if
      `*names` is empty
+    - `skip_scripts`: indicates scripts should be skipped
     """
     log.info(
         "%sInstalling dependencies: %s",
@@ -115,7 +117,7 @@ def install(
         )
         count += _count  # type: ignore
 
-        if _count:
+        if _count and not skip_scripts:
             label = "nested scripts" if index else "scripts"
             common.show(f"Running {label}...", color="message", log=False)
             common.newline()
@@ -136,6 +138,7 @@ def update(
     lock=None,  # pylint: disable=redefined-outer-name
     skip_changes=False,
     skip_default_group=False,
+    skip_scripts=False,
 ):
     """Update dependencies for a project.
 
@@ -155,6 +158,7 @@ def update(
      should be skipped
     - `skip_default_group`: indicates default_group should be skipped if
      `*names` is empty
+    - `skip_scripts`: indicates scripts should be skipped
     """
     log.info(
         "%s dependencies%s: %s",
@@ -198,7 +202,7 @@ def update(
                 skip_changes=skip_changes or force_interactive,
             )
 
-        if _count:
+        if _count and not skip_scripts:
             label = "nested scripts" if index else "scripts"
             common.show(f"Running {label}...", color="message", log=False)
             common.newline()

gitman/plugin.py

@@ -62,6 +62,13 @@ def main(args=None):
         action="store_true",
         dest="no_defaults",
     )
+    parser.add_argument(
+        "-S",
+        "--no-scripts",
+        help="skip running scripts after installation",
+        action="store_true",
+        dest="no_scripts",
+    )
 
     # Update option
     group.add_argument(

gitman/tests/test_cli.py

@@ -77,6 +77,7 @@ def test_install(self, mock_install):
             fetch=False,
             clean=False,
             skip_changes=False,
+            skip_scripts=False,
             skip_default_group=False,
         )
 
@@ -93,6 +94,7 @@ def test_install_root(self, mock_install):
             fetch=False,
             clean=False,
             skip_changes=False,
+            skip_scripts=False,
             skip_default_group=False,
         )
 
@@ -109,6 +111,7 @@ def test_install_force(self, mock_install):
             fetch=False,
             clean=False,
             skip_changes=False,
+            skip_scripts=False,
             skip_default_group=False,
         )
 
@@ -125,6 +128,7 @@ def test_install_fetch(self, mock_install):
             fetch=True,
             clean=False,
             skip_changes=False,
+            skip_scripts=False,
             skip_default_group=False,
         )
 
@@ -141,6 +145,7 @@ def test_install_clean(self, mock_install):
             fetch=False,
             clean=True,
             skip_changes=False,
+            skip_scripts=False,
             skip_default_group=False,
         )
 
@@ -159,6 +164,7 @@ def test_install_specific_sources(self, mock_install):
             fetch=False,
             clean=False,
             skip_changes=False,
+            skip_scripts=False,
             skip_default_group=False,
         )
 
@@ -175,6 +181,7 @@ def test_install_with_depth(self, mock_update):
             fetch=False,
             clean=False,
             skip_changes=False,
+            skip_scripts=False,
             skip_default_group=False,
         )
 
@@ -204,6 +211,7 @@ def test_update(self, mock_update):
             recurse=False,
             lock=None,
             skip_changes=False,
+            skip_scripts=False,
         )
 
     @patch("gitman.commands.update")
@@ -220,6 +228,7 @@ def test_update_recursive(self, mock_update):
             recurse=True,
             lock=None,
             skip_changes=False,
+            skip_scripts=False,
         )
 
     @patch("gitman.commands.update")
@@ -236,6 +245,7 @@ def test_update_no_lock(self, mock_update):
             recurse=False,
             lock=False,
             skip_changes=False,
+            skip_scripts=False,
         )
 
     @patch("gitman.commands.update")
@@ -252,6 +262,7 @@ def test_update_skip_changes(self, mock_update):
             recurse=False,
             lock=None,
             skip_changes=True,
+            skip_scripts=False,
         )
 
     @patch("gitman.commands.update")
@@ -268,6 +279,7 @@ def test_update_force(self, mock_update):
             recurse=False,
             lock=None,
             skip_changes=False,
+            skip_scripts=False,
         )
 
     @patch("gitman.commands.update")
@@ -284,6 +296,7 @@ def test_update_force_interactive(self, mock_update):
             recurse=False,
             lock=None,
             skip_changes=False,
+            skip_scripts=False,
         )
 
     @patch("gitman.commands.update")
@@ -302,6 +315,7 @@ def test_update_specific_sources(self, mock_install):
             recurse=False,
             lock=None,
             skip_changes=False,
+            skip_scripts=False,
         )
 
     @patch("gitman.commands.update")
@@ -318,6 +332,7 @@ def test_update_with_depth(self, mock_update):
             recurse=False,
             lock=None,
             skip_changes=False,
+            skip_scripts=False,
         )
 
 

gitman/tests/test_plugin.py

@@ -6,7 +6,6 @@
 
 
 class TestMain:
-
     """Unit tests for the top-level arguments."""
 
     @patch("gitman.cli.commands")
@@ -25,6 +24,7 @@ def test_install(self, mock_commands):
                 force=False,
                 force_interactive=False,
                 skip_changes=False,
+                skip_scripts=False,
                 skip_default_group=False,
             ),
             call.install().__bool__(),  # command status check
@@ -47,6 +47,7 @@ def test_update(self, mock_commands):
                 recurse=False,
                 lock=True,
                 skip_changes=False,
+                skip_scripts=False,
             ),
             call.update().__bool__(),  # command status check
         ] == mock_commands.mock_calls
@@ -68,6 +69,7 @@ def test_update_recursive(self, mock_commands):
                 recurse=True,
                 lock=True,
                 skip_changes=False,
+                skip_scripts=False,
             ),
             call.update().__bool__(),  # command status check
         ] == mock_commands.mock_calls
@@ -89,6 +91,7 @@ def test_update_no_lock(self, mock_commands):
                 recurse=False,
                 lock=False,
                 skip_changes=False,
+                skip_scripts=False,
             ),
             call.update().__bool__(),  # command status check
         ] == mock_commands.mock_calls
@@ -110,6 +113,7 @@ def test_update_skip_changes(self, mock_commands):
                 recurse=False,
                 lock=True,
                 skip_changes=True,
+                skip_scripts=False,
             ),
             call.update().__bool__(),  # command status check
         ] == mock_commands.mock_calls

pyproject.toml

@@ -1,7 +1,7 @@
 [tool.poetry]
 
 name = "gitman"
-version = "3.4.1b1"
+version = "3.4.1b2"
 description = "A language-agnostic dependency manager using Git."
 
 license = "MIT"