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 8eb7d97e65 Fix passing packages as arguments to build-docs (#37646)
8eb7d97e65 is described below
commit 8eb7d97e655211ae4cfe3174c70e5511b0070389
Author: Amogh Desai <[email protected]>
AuthorDate: Fri Feb 23 11:38:48 2024 +0530
Fix passing packages as arguments to build-docs (#37646)
---
dev/breeze/src/airflow_breeze/commands/developer_commands.py | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/dev/breeze/src/airflow_breeze/commands/developer_commands.py
b/dev/breeze/src/airflow_breeze/commands/developer_commands.py
index a781cacd07..3524ded280 100644
--- a/dev/breeze/src/airflow_breeze/commands/developer_commands.py
+++ b/dev/breeze/src/airflow_breeze/commands/developer_commands.py
@@ -650,17 +650,17 @@ def build_docs(
for directory in docs_dir.rglob(dir_name):
get_console().print(f"[info]Removing {directory}")
shutil.rmtree(directory, ignore_errors=True)
- temp_doc_packages = None
+ docs_list_as_tuple: tuple[str, ...] = ()
if package_list and len(package_list):
get_console().print(f"\n[info]Populating provider list from
PACKAGE_LIST env as {package_list}")
# Override doc_packages with values from PACKAGE_LIST
- temp_doc_packages = tuple(package_list.split(","))
- if doc_packages and package_list:
+ docs_list_as_tuple = tuple(package_list.split(","))
+ if doc_packages and docs_list_as_tuple:
get_console().print(
f"[warning]Both package arguments and --package-list /
PACKAGE_LIST passed. "
- f"Overriding to {temp_doc_packages}"
+ f"Overriding to {docs_list_as_tuple}"
)
- doc_packages = temp_doc_packages or ()
+ doc_packages = docs_list_as_tuple or doc_packages
doc_builder = DocBuildParams(
package_filter=package_filter,
docs_only=docs_only,
