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

pierrejeambrun 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 5e78aaa28e Check _is_canary_run/pr condition in 
is_legacy_ui_api_labeled method (#42921)
5e78aaa28e is described below

commit 5e78aaa28e1b8a8d901458ec4cc18d044483db51
Author: GPK <[email protected]>
AuthorDate: Fri Oct 11 09:19:37 2024 +0100

    Check _is_canary_run/pr condition in is_legacy_ui_api_labeled method 
(#42921)
    
    * check _is_canary_run condition in is_legacy_ui_api_labeled method
    
    * include pr check in is_legacy_ui_api_labeled
---
 dev/breeze/src/airflow_breeze/utils/selective_checks.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py 
b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
index dfc24993f2..18458af5f8 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -1370,6 +1370,12 @@ class SelectiveChecks:
     def is_legacy_ui_api_labeled(self) -> bool:
         # Selective check for legacy UI/API updates.
         # It is to ping the maintainer to add the label and make them aware of 
the changes.
+        if self._is_canary_run() or self._github_event not in (
+            GithubEvents.PULL_REQUEST,
+            GithubEvents.PULL_REQUEST_TARGET,
+        ):
+            return False
+
         if (
             self._matching_files(
                 FileGroupForCi.LEGACY_API_FILES, CI_FILE_GROUP_MATCHES, 
CI_FILE_GROUP_EXCLUDES

Reply via email to