Merge pull request #114 from petterroea/master

Fix not being able to tag without also committing
callowayproject · Dec 29, 2023 · df4ef17 · df4ef17
2 parents 9418a14 + 753c990
commit df4ef17
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/bumpversion/scm.py b/bumpversion/scm.py
@@ -177,9 +177,6 @@ def commit_to_scm(
     @classmethod
     def tag_in_scm(cls, config: "Config", context: MutableMapping, dry_run: bool = False) -> None:
         """Tag the current commit in the source code management system."""
-        if not config.commit:
-            logger.info("Would not tag since we are not committing")
-            return
         if not config.tag:
             logger.info("Would not tag")
             return

diff --git a/tests/test_scm.py b/tests/test_scm.py
@@ -160,7 +160,7 @@ def test_commit_and_tag_from_below_scm_root(repo: str, scm_command: str, scm_cla
     [
         param(True, True, True, False, False, id="dry-run-stops-commit-and-tag"),
         param(True, False, False, True, False, id="commit-no-tag"),
-        param(False, True, False, False, False, id="no-commit-stops-tag"),
+        param(False, True, False, False, True, id="no-commit-will-tag"),
     ],
 )
 def test_commit_tag_dry_run_interactions(