This is an automated email from the ASF dual-hosted git repository.

sbp pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-releases.git


The following commit(s) were added to refs/heads/main by this push:
     new 28a3654  Fix a bug in the file interface
28a3654 is described below

commit 28a365475557b4646ea0b50e247ac5016a24a35e
Author: Sean B. Palmer <[email protected]>
AuthorDate: Tue Dec 16 15:50:19 2025 +0000

    Fix a bug in the file interface
---
 atr/get/file.py | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/atr/get/file.py b/atr/get/file.py
index bb56d9d..85f7877 100644
--- a/atr/get/file.py
+++ b/atr/get/file.py
@@ -40,16 +40,15 @@ async def selected(session: web.Committer, project_name: 
str, version_name: str)
 
     revision_number = release.latest_revision_number
     file_stats = []
-    if revision_number is not None:
-        if release.phase == sql.ReleasePhase.RELEASE:
-            file_stats = [stat async for stat in 
util.content_list(util.get_finished_dir(), project_name, version_name)]
-        else:
-            file_stats = [
-                stat
-                async for stat in util.content_list(
-                    util.get_unfinished_dir(), project_name, version_name, 
revision_number
-                )
-            ]
+    if release.phase == sql.ReleasePhase.RELEASE:
+        file_stats = [stat async for stat in 
util.content_list(util.get_finished_dir(), project_name, version_name)]
+    elif revision_number is not None:
+        file_stats = [
+            stat
+            async for stat in util.content_list(util.get_unfinished_dir(), 
project_name, version_name, revision_number)
+        ]
+    else:
+        raise ValueError("No revision number found for unfinished release")
     file_stats.sort(key=lambda fs: fs.path)
 
     block = htm.Block()


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to