diff --git a/newsfragments/46231.significant.rst b/newsfragments/46231.significant.rst index 8d9081c1c692a..9d0ab34a438e2 100644 --- a/newsfragments/46231.significant.rst +++ b/newsfragments/46231.significant.rst @@ -1,6 +1,6 @@ -.. Moving EmptyOperator from Airflow core to the ``standard`` provider. +Moving EmptyOperator from Airflow core to the ``standard`` provider. -.. EmptyOperator has been moved from Airflow core (``airflow.operators directory``) to the ``standard`` provider. +EmptyOperator has been moved from Airflow core (``airflow.operators directory``) to the ``standard`` provider. For new and existing DAGs, users must import ``EmptyOperator`` from ``airflow.providers.standard.operators.empty``. * Types of change @@ -14,8 +14,6 @@ For new and existing DAGs, users must import ``EmptyOperator`` from ``airflow.pr * [ ] Dependency changes * [ ] Code interface changes -.. List the migration rules needed for this change (see https://github.com/apache/airflow/issues/41641) - * Migration rules needed * ruff diff --git a/newsfragments/46375.significant.rst b/newsfragments/46375.significant.rst index fd3a91ad7157b..b8443726d4120 100644 --- a/newsfragments/46375.significant.rst +++ b/newsfragments/46375.significant.rst @@ -19,4 +19,4 @@ Any occurrences of the ``secrets_masker`` module will have to be updated from `` * AIR302 - * [ ] ``airflow.utils.log.secrets_masker`` -> ``airflow.sdk.execution_time.secrets_masker`` + * [ ] ``airflow.utils.log.secrets_masker`` → ``airflow.sdk.execution_time.secrets_masker`` diff --git a/newsfragments/46408.significant.rst b/newsfragments/46408.significant.rst index 5088f81d4d8b1..dc04bd4d701a9 100644 --- a/newsfragments/46408.significant.rst +++ b/newsfragments/46408.significant.rst @@ -25,6 +25,6 @@ If these config options are still present, they will have no effect any longer. * ``airflow config lint`` - * Remove ``[logging] dag_processor_manager_log_location`` - * Remove ``[logging] dag_processor_manager_log_stdout`` - * Remove ``[logging] log_processor_filename_template`` + * [ ] ``[logging] dag_processor_manager_log_location`` + * [ ] ``[logging] dag_processor_manager_log_stdout`` + * [ ] ``[logging] log_processor_filename_template`` diff --git a/scripts/ci/pre_commit/significant_newsfragments_checker.py b/scripts/ci/pre_commit/significant_newsfragments_checker.py index afd13bb2d9eac..61a4bcbe02b6b 100755 --- a/scripts/ci/pre_commit/significant_newsfragments_checker.py +++ b/scripts/ci/pre_commit/significant_newsfragments_checker.py @@ -45,7 +45,7 @@ {% endfor %} {%- endfor %} {%- endif -%} -{% if undone_config_rules -%} +{%- if undone_config_rules %} ======airflow config lint rules====== {% for rule in undone_config_rules %}* {{ rule }} {% endfor %} @@ -148,6 +148,10 @@ def _extract_check_list_item(node: docutils.nodes.Node) -> tuple[bool, str]: raise ValueError(f"Incorrect format {node.astext()}") text = node.astext() + if text[0] != "[" or text[2] != "]": + raise ValueError( + f"{text} should be a checklist (e.g., * [ ] ``logging.dag_processor_manager_log_location``)" + ) return text[:3] == "[x]", text[4:] @property