fix: capture slack errors without throwing

CHANGELOG.md

@@ -6,6 +6,7 @@
 - Adds a healthcheck to the nginx Docker container
 - Prints deployment log to the error logger if `log_level` is set to `DEBUG`
 - Raises a `HarveyError` on deployment fail instead of `sys.exit(1)`, removes redundant `sys.exit(0)` on success
+- Captures Slack errors via Sentry but no longer raises an exception to ensure builds continue even if there is a problem with message delivery
 
 ## v1.0.3 (2023-10-25)
 

harvey/messages.py

@@ -1,8 +1,8 @@
 import slack_sdk
 import woodchips
+from sentry_sdk import capture_exception
 
 from harvey.config import Config
-from harvey.errors import HarveyError
 
 
 class Message:
@@ -23,7 +23,7 @@ def send_slack_message(message: str):
                 text=message,
             )
             logger.debug('Slack message sent!')
-        except slack_sdk.errors.SlackApiError:
-            error_message = 'Harvey could not send the Slack message.'
+        except slack_sdk.errors.SlackApiError as error:
+            error_message = f'Harvey could not send the Slack message: {error}'
             logger.error(error_message)
-            raise HarveyError(error_message)
+            capture_exception(error_message)