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-release.git


The following commit(s) were added to refs/heads/main by this push:
     new 0e9c840  Fix misordered interfaces in the API client script
0e9c840 is described below

commit 0e9c8403e072da74ad8b57eec4aaec314d7af1c1
Author: Sean B. Palmer <[email protected]>
AuthorDate: Mon Jul 7 19:29:00 2025 +0100

    Fix misordered interfaces in the API client script
---
 client/atr | 50 +++++++++++++++++++++++++-------------------------
 1 file changed, 25 insertions(+), 25 deletions(-)

diff --git a/client/atr b/client/atr
index 248dbda..dfc7e97 100755
--- a/client/atr
+++ b/client/atr
@@ -115,7 +115,7 @@ def app_checks_status(project: str, version: str, revision: 
str) -> None:
     verify_ssl = not host.startswith("127.0.0.1")
     results = asyncio.run(web_get(url, jwt_value, verify_ssl))
 
-    check_results_display(results)
+    checks_display(results)
 
 
 @CONFIG.command(name="file", help="Display the configuration file contents.")
@@ -330,6 +330,27 @@ def app_test(q: Annotated[bool, 
cyclopts.Parameter(alias="-q")] = False, *pytest
             os.chdir(cwd)
 
 
+def checks_display(results: list[dict[str, Any]]) -> None:
+    if not results:
+        print("No check results found for this revision.")
+        return
+
+    by_status = {}
+    for result in results:
+        status = result["status"]
+        by_status.setdefault(status, []).append(result)
+
+    print(f"Total checks: {len(results)}")
+    for status, checks in by_status.items():
+        print(f"  {status}: {len(checks)}")
+
+    for status in ["FAILURE", "EXCEPTION", "WARNING"]:
+        if status in by_status:
+            print(f"\n{status}:")
+            for check in by_status[status]:
+                print(f"  - {check['checker']}: {check['message']}")
+
+
 def config_drop(config: dict[str, Any], parts: list[str]) -> None:
     config_walk(config, parts, "drop")
 
@@ -415,25 +436,9 @@ def config_write(data: dict[str, Any]) -> None:
     os.replace(tmp, path)
 
 
-def check_results_display(results: list[dict[str, Any]]) -> None:
-    if not results:
-        print("No check results found for this revision.")
-        return
-
-    by_status = {}
-    for result in results:
-        status = result["status"]
-        by_status.setdefault(status, []).append(result)
-
-    print(f"Total checks: {len(results)}")
-    for status, checks in by_status.items():
-        print(f"  {status}: {len(checks)}")
-
-    for status in ["FAILURE", "EXCEPTION", "WARNING"]:
-        if status in by_status:
-            print(f"\n{status}:")
-            for check in by_status[status]:
-                print(f"  - {check['checker']}: {check['message']}")
+def main() -> None:
+    logging.basicConfig(level=logging.INFO, format="%(message)s")
+    APP()
 
 
 def releases_display(result: dict[str, Any]) -> None:
@@ -464,11 +469,6 @@ def releases_display(result: dict[str, Any]) -> None:
         print(f"  {version:<24} {phase_short:<11} {created_formatted}")
 
 
-def main() -> None:
-    logging.basicConfig(level=logging.INFO, format="%(message)s")
-    APP()
-
-
 def test_app_release_list_not_found(
     capsys: pytest.CaptureFixture[str], monkeypatch: pytest.MonkeyPatch, 
tmp_path: pathlib.Path
 ) -> None:


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

Reply via email to