supervisor/jira_utils.py: Fix a dry-run bug in attachment upload function

owtaylor · owtaylor · commit 331b24a7e3bd · 2025-10-27T10:38:59.000-04:00
I accidentally removed the early return value when changing the return value
 from list[str] to None.
diff --git a/supervisor/jira_utils.py b/supervisor/jira_utils.py
@@ -615,6 +615,7 @@ def add_issue_attachments(
             issue_key,
         )
         logger.debug("Dry run: would post attachment to %s", path)
+        return
 
     jira_api_upload(path, attachments=attachments)
 

Original file line number	Diff line number	Diff line change
`@@ -615,6 +615,7 @@ def add_issue_attachments(`
`615`	`615`	`issue_key,`
`616`	`616`	`)`
`617`	`617`	`logger.debug("Dry run: would post attachment to %s", path)`
	`618`	`+ return`
`618`	`619`
`619`	`620`	`jira_api_upload(path, attachments=attachments)`
`620`	`621`