github-actions[bot] opened a new pull request, #75102: URL: https://github.com/apache/airflow/pull/75102
* Validate CallbackTrigger callback path before running the callback The callback path resolved by CallbackTrigger must now be a coroutine function or an awaitable callable (such as an async notifier class), matching what AsyncCallback accepts. Anything else fails the callback before it is called, the same way trigger classpaths are validated. Generated-by: Claude Opus 5 following the guidelines at https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions * Rename newsfragment to the PR number Generated-by: Claude Opus 5 following the guidelines at https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions * Accept the same triggerer callbacks the Task SDK accepts The triggerer also accepted callable objects with an async __call__, which AsyncCallback rejects when the callback is defined, so the two checks disagreed. Core cannot import the SDK check, so the triggerer now applies the same rule locally. (cherry picked from commit 57fb394afe88a077074914d76db572f68856c0bb) Co-authored-by: Jarek Potiuk <[email protected]> Generated-by: Claude Opus 5 -- 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]
