potiuk commented on code in PR #37057:
URL: https://github.com/apache/airflow/pull/37057#discussion_r1468880404
##########
dev/breeze/src/airflow_breeze/utils/selective_checks.py:
##########
@@ -1039,3 +1040,9 @@ def providers_compatibility_checks(self) -> str:
if check["python-version"] in self.python_versions
]
)
+
+ @cached_property
+ def is_committer_build(self):
+ if NON_COMMITTER_BUILD_LABEL in self._pr_labels:
+ return False
+ return self._github_actor in COMMITTERS
Review Comment:
OK. I applied a number of cleanups and clairifcations and comments in the
workflow describing what's going on. I also found out along the way that the
part where DEFAULT_BRANCH and DEFAULT_CONSTRAINTS_BRANCH retrieval can be
simplified, after some past changes in the way how "pull request workflow" uses
now target_branhc by default and not "main" as it used to be, so we do not have
to use evn variables to pass it now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
