This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch v2-8-test in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 250b0af4607508cac10444d16d389473b6093685 Author: Amogh Desai <[email protected]> AuthorDate: Sun Feb 18 01:11:58 2024 +0530 Fixing an edge case of breeze publish-docs summary (#37506) (cherry picked from commit 65642ed9503f6135f781ac5e5158d375fb852d4a) --- .../commands/release_management_commands.py | 42 +++++++++++++--------- 1 file changed, 26 insertions(+), 16 deletions(-) diff --git a/dev/breeze/src/airflow_breeze/commands/release_management_commands.py b/dev/breeze/src/airflow_breeze/commands/release_management_commands.py index 50ac9cdac5..c2b5766c93 100644 --- a/dev/breeze/src/airflow_breeze/commands/release_management_commands.py +++ b/dev/breeze/src/airflow_breeze/commands/release_management_commands.py @@ -1238,14 +1238,19 @@ def run_publish_docs_in_parallel( else: skipped_entries.append(message) - get_console().print("[blue]Summary:\n") - get_console().print("[success]Packages published:") - for entry in success_entries: - get_console().print(f"[success]{entry}") - get_console().rule() - get_console().print("\n[warning]Packages skipped:") - for entry in skipped_entries: - get_console().print(f"[warning]{entry}") + get_console().print("[blue]Summary:") + need_rule = False + if len(success_entries): + get_console().print("[success]Packages published:") + for entry in success_entries: + get_console().print(f"[success]{entry}") + need_rule = True + if need_rule: + get_console().rule() + if len(skipped_entries): + get_console().print("\n[warning]Packages skipped:") + for entry in skipped_entries: + get_console().print(f"[warning]{entry}") @release_management.command( @@ -1325,14 +1330,19 @@ def publish_docs( success_entries.append(message) else: skipped_entries.append(message) - get_console().print("[blue]Summary:\n") - get_console().print("[success]Packages published:") - for entry in success_entries: - get_console().print(f"[success]{entry}") - get_console().rule() - get_console().print("\n[warning]Packages skipped:") - for entry in skipped_entries: - get_console().print(f"[warning]{entry}") + get_console().print("[blue]Summary:") + need_rule = False + if len(success_entries): + get_console().print("[success]Packages published:") + for entry in success_entries: + get_console().print(f"[success]{entry}") + need_rule = True + if need_rule: + get_console().rule() + if len(skipped_entries): + get_console().print("\n[warning]Packages skipped:") + for entry in skipped_entries: + get_console().print(f"[warning]{entry}") @release_management.command(
