This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 65642ed950 Fixing an edge case of breeze publish-docs summary (#37506)
65642ed950 is described below
commit 65642ed9503f6135f781ac5e5158d375fb852d4a
Author: Amogh Desai <amoghrajesh1...@gmail.com>
AuthorDate: Sun Feb 18 01:11:58 2024 +0530
Fixing an edge case of breeze publish-docs summary (#37506)
---
.../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 3edeed633a..43b2570ad0 100644
--- a/dev/breeze/src/airflow_breeze/commands/release_management_commands.py
+++ b/dev/breeze/src/airflow_breeze/commands/release_management_commands.py
@@ -1244,14 +1244,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(
@@ -1331,14 +1336,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(
