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

potiuk 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 ec0113b71a Move instructions about the link to docs to after all 
packages (#35467)
ec0113b71a is described below

commit ec0113b71ad8142bc45d092cab9790e6602eb5f0
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon Nov 6 08:44:09 2023 +0100

    Move instructions about the link to docs to after all packages (#35467)
    
    The instructions were printed after every package, but it makes
    sense to print them only after all packages are complte.
---
 dev/breeze/src/airflow_breeze/commands/developer_commands.py | 4 ++++
 docs/exts/docs_build/docs_builder.py                         | 4 ----
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/dev/breeze/src/airflow_breeze/commands/developer_commands.py 
b/dev/breeze/src/airflow_breeze/commands/developer_commands.py
index 761a23aed2..60d3138e68 100644
--- a/dev/breeze/src/airflow_breeze/commands/developer_commands.py
+++ b/dev/breeze/src/airflow_breeze/commands/developer_commands.py
@@ -460,6 +460,10 @@ def build_docs(
         *doc_builder.args_doc_builder,
     ]
     process = run_command(cmd, text=True, env=env, check=False)
+    if process.returncode == 0:
+        get_console().print(
+            "[info]Start the webserver in breeze and view the built docs at 
http://localhost:28080/docs/[/]";
+        )
     sys.exit(process.returncode)
 
 
diff --git a/docs/exts/docs_build/docs_builder.py 
b/docs/exts/docs_build/docs_builder.py
index 446aaddd43..375e9db8c5 100644
--- a/docs/exts/docs_build/docs_builder.py
+++ b/docs/exts/docs_build/docs_builder.py
@@ -263,10 +263,6 @@ class AirflowDocsBuilder:
             console.print(f"[info]{self.package_name:60}:[/] [red]Finished 
docs building with errors[/]")
         else:
             console.print(f"[info]{self.package_name:60}:[/] [green]Finished 
docs building successfully[/]")
-            console.print(
-                f"[info]{self.package_name:60}:[/] [green]Start the webserver 
in breeze and view "
-                f"the built docs at http://localhost:28080/docs/[/]";
-            )
         return build_errors
 
     def publish(self, override_versioned: bool):

Reply via email to