fix: warn instead of exception for nothing to commit

Author: Justintime50

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## Next Release
 
 - Adds `custom_tarball` parameter, allowing you to bundle your distributable assets however you'd like (closes #63)
+- Homebrew Releaser now warns instead of raising an exception and exiting with a failure if the `git commit` operation is unsuccessful due to "nothing to commit". This is useful if you want to test back to back releases or need to recreate a release but the underlying assets have not changed (closes #69)
 - Optimizes how asset URL selection occurs when downloading assets to generate checksums for
 
 ## v3.1.0 (2025-12-26)

homebrew_releaser/app.py

@@ -227,6 +227,7 @@ def run_github_action():
         logger.debug("Skipping update to project README.")
 
     # Although users can skip a commit, still commit (but don't push) to dry-run the commit for debugging purposes
+    logger.info("Preparing git commit...")
     copy_formula_file_to_git(formula_filepath, HOMEBREW_TAP)
     add_git(HOMEBREW_TAP)
     commit_git(HOMEBREW_TAP, GITHUB_REPO, version)

homebrew_releaser/git.py

@@ -63,10 +63,17 @@ def add_git(homebrew_tap: str):
 
 def commit_git(homebrew_tap: str, repo_name: str, version: str):
     """Commits git assets to the Homebrew tap repo."""
+    logger = woodchips.get(LOGGER_NAME)
     # fmt: off
     command = ['git', '-C', build_dir_path(homebrew_tap), 'commit', '-m', f'chore: brew formula update for {repo_name} {version}']  # noqa
     # fmt: on
-    _run_git_subprocess(command, "Assets committed successfully.")
+    try:
+        _run_git_subprocess(command, "Assets committed successfully.")
+    except subprocess.CalledProcessError as e:
+        if "nothing to commit" in e.output:
+            logger.warning("No changes to commit.")
+        else:
+            raise
 
 
 def push_git(homebrew_tap: str, homebrew_owner: str):

test/unit/test_git.py

@@ -104,6 +104,32 @@ def test_commit(mock_subprocess):
     )
 
 
+@patch("homebrew_releaser.utils.WORKING_DIR", "")
+@patch("logging.Logger.warning")
+@patch("subprocess.check_output")
+def test_nothing_to_commit(mock_subprocess, mock_logger):
+    """Tests that we call the correct git commit command."""
+    homebrew_tap = "homebrew-formulas"
+    repo_name = "mock-repo"
+    version = "0.1.0"
+
+    error = subprocess.CalledProcessError(
+        returncode=1,
+        cmd=["git", "-C", homebrew_tap, "commit", "-m", f"chore: brew formula update for {repo_name} {version}"],
+        output="On branch main\nnothing to commit, working tree clean\n",
+    )
+    mock_subprocess.side_effect = error
+
+    commit_git(homebrew_tap, repo_name, version)
+    mock_subprocess.assert_called_once_with(
+        ["git", "-C", homebrew_tap, "commit", "-m", f"chore: brew formula update for {repo_name} {version}"],
+        stderr=-2,
+        text=True,
+        timeout=TIMEOUT,
+    )
+    mock_logger.assert_called_once_with("No changes to commit.")
+
+
 @patch("homebrew_releaser.utils.WORKING_DIR", "")
 @patch("homebrew_releaser.git.GITHUB_TOKEN", "123")
 @patch("subprocess.check_output")