sts-sponsors team mailing list archive

[Alberto Donato <mp+442256@xxxxxxxxxxxxxxxxxx>]

Alberto Donato has proposed merging ~ack/maas/+git/maas-release-tools:release-status-checks-before into ~maas-committers/maas/+git/maas-release-tools:main.

Commit message:
print out check status before the text (so they're aligned)



Requested reviews:
  MAAS Lander (maas-lander): unittests
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~ack/maas/+git/maas-release-tools/+merge/442256
-- 
Your team MAAS Committers is requested to review the proposed merge of ~ack/maas/+git/maas-release-tools:release-status-checks-before into ~maas-committers/maas/+git/maas-release-tools:main.

diff --git a/maas_release_tools/scripts/release_status.py b/maas_release_tools/scripts/release_status.py
index 67ecb57..260c5f2 100644
--- a/maas_release_tools/scripts/release_status.py
+++ b/maas_release_tools/scripts/release_status.py
@@ -126,16 +126,18 @@ class ReleasePreparer:
             how_to_fix = None
             if step.skip():
                 continue
-            print(step.title, end=": ")
+            print(f"\N{Radio Button} {step.title}", end="", flush=True)
             success, message = step.check()
             if not success:
                 success, how_to_fix = step.fix()
 
             if not success:
                 all_good = False
-                print("\N{Cross Mark}")
+                marker = "\N{Cross Mark}"
             else:
-                print("\N{White Heavy Check Mark}")
+                marker = "\N{White Heavy Check Mark}"
+            # rewrite the line once step is done
+            print(f"\r{marker} {step.title}")
 
             if message:
                 for line in message.splitlines():