github-actions[bot] opened a new pull request, #72898: URL: https://github.com/apache/airflow/pull/72898
The check only recognized a Dag constructor when the callable resolved through a from-import of airflow's DAG or dag. A Dag built from a subclass, or reached through an aliased module import, produced no warnings at all — and because task detection keys off the same predicate, neither did any task inside its with-block. The Dag file is not imported yet when the check runs, so there is no object to test — only names and the file's own imports. The cost of leaning on names is an occasional false positive on something merely named like a Dag. (cherry picked from commit 1e2ad803f74eadb5fb28d4504ca06b008c48b63e) Co-authored-by: Jed Cunningham <[email protected]> -- 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]
