sejal-gupta-ksolves commented on PR #67327: URL: https://github.com/apache/airflow/pull/67327#issuecomment-4532453196
> @sejal-gupta-ksolves A few things need addressing before review — see our [Pull Request quality criteria](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-quality-criteria). > > * ❌ **Provider compat tests fail across 3 Airflow versions**: `Compat 3.0.6:P3.10:`, `Compat 3.1.8:P3.10:`, `Compat 3.2.1:P3.10:` — 7 failing checks total. The change may rely on a symbol or behaviour not present in older supported Airflow versions. See the [Checks tab](https://github.com/apache/airflow/pull/67327/checks). > > No rush. > > _Note: This comment was drafted by an AI-assisted triage tool and may contain mistakes. Once you have addressed the points above, an Apache Airflow maintainer — a real person — will take the next look at your PR. We use this [two-stage triage process](https://github.com/apache/airflow/blob/main/contributing-docs/25_maintainer_pr_triage.md#why-the-first-pass-is-automated) so that our maintainers' limited time is spent where it matters most: the conversation with you._ > > Drafted-by: Claude Code (Opus 4.7); reviewed by @potiuk before posting Hi @potiuk that makes total sense **build_airflow_dagrun_url** isn't around in those earlier Airflow 3 minor versions. I've wrapped the lazy import and XCom push in a **try/except ImportError** block to keep it completely backwards compatible. The PR has been updated! -- 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]
