potiuk commented on PR #67341: URL: https://github.com/apache/airflow/pull/67341#issuecomment-4529375475
@Prab-27 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). - :x: **15 failing checks across shared package tests** (e.g. `Basic tests / Shared providers_discovery tests`, `Basic tests / Shared secrets_backend tests`, `Basic tests / Shared observability tests`). A pandas dependency upgrade in a single provider shouldn't normally break shared-package tests — worth verifying the diff is scoped correctly and that the pandas bump doesn't drag in incompatible transitive deps. See the [Checks tab](https://github.com/apache/airflow/pull/67341/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 -- 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]
