fix: error message is a string, not a tuple

dimaqq · web-flow · commit 656a511c72a0 · 2025-02-20T16:40:35.000+09:00
I imagine someone refactored and moved message out int a variable, but forgot that in that case comma after argument is significant. Fixes #339
diff --git a/attestations.py b/attestations.py
@@ -83,7 +83,7 @@ def assert_attestations_do_not_pre_exist(
     existing_attestations_list = '\n'.join(map(str, existing_attestations))
     error_message = (
         'The following distributions already have publish attestations:'
-        f'{existing_attestations_list}',
+        f'{existing_attestations_list}'
     )
     die(error_message)
 

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ def assert_attestations_do_not_pre_exist(`
`83`	`83`	`existing_attestations_list = '\n'.join(map(str, existing_attestations))`
`84`	`84`	`error_message = (`
`85`	`85`	`'The following distributions already have publish attestations:'`
`86`		`- f'{existing_attestations_list}',`
	`86`	`+ f'{existing_attestations_list}'`
`87`	`87`	`)`
`88`	`88`	`die(error_message)`
`89`	`89`