This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch v2-6-test in repository https://gitbox.apache.org/repos/asf/airflow.git
commit bdda26342363ec26371e8131027ad46cfeced288 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. (cherry picked from commit de585f521b5898ba7687072a7717fd3b67fa8c5c) --- 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 ad8ffadfa5..ba9f015437 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",), {
