fix: existing checksum now warns instead of providing a stacktrace

Author: Justintime50

CHANGELOG.md

@@ -5,6 +5,7 @@
 - Adds `custom_tarball` parameter, allowing you to bundle your distributable assets however you'd like (closes #63)
 - Adds `branch` parameter, allowing you to push the formula file to any branch you'd like (closes #62)
 - 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)
+- Existing `checksum.txt` files on the latest release now warn if they exist instead of exiting with a stacktrace
 - Optimizes how asset URL selection occurs when downloading assets to generate checksums for
 - Makes `formula_folder` optional as we always had a default of `Formula`, now allows users to omit its inclusion in their yaml
 - Bumps Homebrew to v5.0.9

homebrew_releaser/checksum.py

@@ -26,7 +26,7 @@ def calculate_checksum(tar_filepath: str) -> str:
     return checksum
 
 
-def upload_checksum_file(latest_release: dict[str, Any]):
+def upload_checksum_file(latest_release: dict[str, Any]) -> None:
     """Uploads a `checksum.txt` file to the latest release of the repo."""
     logger = woodchips.get(LOGGER_NAME)
 
@@ -39,11 +39,17 @@ def upload_checksum_file(latest_release: dict[str, Any]):
     headers = GITHUB_HEADERS.copy()
     headers["Content-Type"] = "text/plain"
 
-    response = requests.post(
-        upload_url,
-        headers=headers,
-        data=checksum_file_content,
-        timeout=TIMEOUT,
-    )
-    response.raise_for_status()
-    logger.info(f"checksum.txt uploaded successfully to {GITHUB_REPO}.")
+    try:
+        response = requests.post(
+            upload_url,
+            headers=headers,
+            data=checksum_file_content,
+            timeout=TIMEOUT,
+        )
+        response.raise_for_status()
+        logger.info(f"checksum.txt uploaded successfully to {GITHUB_REPO}.")
+    except requests.HTTPError:
+        if response.status_code == 422 and "already_exists" in response.text:
+            logger.warning("checksum.txt already exists in the latest release.")
+            return None
+        raise

homebrew_releaser/readme_updater.py

@@ -141,9 +141,9 @@ def _retrieve_old_table(homebrew_tap: str) -> Tuple[str, bool]:
         if old_table_found is False:
             # If we can't find both start/end tags, reset the table so we don't blow away unassociated README data
             old_table = ""
-            logger.error("Could not find both start and end tags for project table in README.")
+            logger.warning("Could not find both start and end tags for project table in README.")
     else:
-        logger.error("Could not find a valid README in this project to update.")
+        logger.warning("Could not find a valid README in this project to update.")
 
     return old_table, old_table_found