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 de585f521b Add airflow to docs packages if only docs/apache-airflow/*
files changed (#32149)
de585f521b is described below
commit de585f521b5898ba7687072a7717fd3b67fa8c5c
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon Jun 26 18:02:26 2023 +0200
Add airflow to docs packages if only docs/apache-airflow/* files changed
(#32149)
Selective checks for docs building missed the case where airflow
package docs should also be build when docs/appache-airflow/*
files changed.
---
dev/breeze/src/airflow_breeze/utils/selective_checks.py | 4 +++-
dev/breeze/tests/test_selective_checks.py | 16 ++++++++++++++++
2 files changed, 19 insertions(+), 1 deletion(-)
diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
index 6aa79df7cb..c6e414472d 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -723,7 +723,9 @@ class SelectiveChecks:
):
return _ALL_DOCS_LIST
packages = []
- if any([file.startswith("airflow/") for file in self._files]):
+ if any(
+ [file.startswith("airflow/") or
file.startswith("docs/apache-airflow/") for file in self._files]
+ ):
packages.append("apache-airflow")
if any([file.startswith("chart/") or
file.startswith("docs/helm-chart") for file in self._files]):
packages.append("helm-chart")
diff --git a/dev/breeze/tests/test_selective_checks.py
b/dev/breeze/tests/test_selective_checks.py
index 4869d70739..dac17ce815 100644
--- a/dev/breeze/tests/test_selective_checks.py
+++ b/dev/breeze/tests/test_selective_checks.py
@@ -1014,6 +1014,22 @@ def test_upgrade_to_newer_dependencies(files: tuple[str,
...], expected_outputs:
},
id="Airbyte provider docs changed",
),
+ pytest.param(
+ ("docs/apache-airflow-providers-airbyte/docs.rst",
"docs/apache-airflow/docs.rst"),
+ {
+ "docs-filter-list-as-string": "--package-filter apache-airflow
"
+ "--package-filter apache-airflow-providers-airbyte "
+ "--package-filter apache-airflow-providers-http",
+ },
+ id="Airbyte provider and airflow core docs changed",
+ ),
+ pytest.param(
+ ("docs/apache-airflow/docs.rst",),
+ {
+ "docs-filter-list-as-string": "--package-filter
apache-airflow",
+ },
+ id="Only Airflow docs changed",
+ ),
pytest.param(
("airflow/providers/celery/file.py",),
{
