Report the number of incomplete publish targets on failure (#66)

src/packse/publish.py

@@ -4,6 +4,7 @@
 import logging
 import os
 import subprocess
+import sys
 import textwrap
 import time
 from concurrent.futures import ThreadPoolExecutor
@@ -72,8 +73,21 @@ def publish(
 
         wait_for_futures(futures)
 
-    results = [future.result() for future in futures]
-    for result in sorted(results):
+    # All the futures are done since the executor context has exited; check if any failed
+    incomplete = 0
+    for future in futures:
+        if (exc := future.exception()) is None:
+            continue
+        incomplete += 1
+        print(f"{exc}.", file=sys.stderr)
+
+    if incomplete:
+        s = "" if targets == 1 else "s"
+        raise PublishError(
+            f"Failed to publish {len(targets) - incomplete}/{len(targets)} target{s}"
+        )
+
+    for result in sorted(future.result() for future in futures):
         print(result)
 
 

tests/__snapshots__/test_publish.ambr

@@ -108,6 +108,7 @@
       Publishing 1 target to https://test.pypi.org/legacy/...
       Publishing 'example-97db95eb'...
       Publish for 'example_97db95eb-0.0.0.tar.gz' already exists.
+      Failed to publish 0/1 targets.
 
     ''',
     'stdout': '',
@@ -120,6 +121,7 @@
       Publishing 1 target to https://test.pypi.org/legacy/...
       Publishing 'example-97db95eb'...
       Publish of 'example_97db95eb-0.0.0.tar.gz' failed due to rate limits.
+      Failed to publish 0/1 targets.
 
     ''',
     'stdout': '',
@@ -134,6 +136,7 @@
       Publishing example_97db95eb-0.0.0.tar.gz with twine failed:
           <twine error message>
       .
+      Failed to publish 0/1 targets.
 
     ''',
     'stdout': '',