potiuk commented on PR #65466: URL: https://github.com/apache/airflow/pull/65466#issuecomment-4299470270
@shan-zeeshan786 This PR has been converted to **draft** because it does not yet meet our [Pull Request quality criteria](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-quality-criteria). **Issues found:** - :x: **Failing CI**: seven checks are failing across several provider test matrices: `MySQL / DB-providers`, `Sqlite / DB-providers`, `Postgres / DB-providers`, `Lowest direct dependency providers tests`, `Non-DB tests / providers` — plus `Build documentation` and `spellcheck-only`. The broad provider impact suggests the serialization change touches a field that providers depend on. Reproduce locally with `breeze testing providers-tests --run-in-parallel` and investigate the failing DB-providers tests first to find the root cause; the docs failures are likely separate and easy to chase by running `breeze build-docs`. **What to do next:** - Fix the serialization regression and push. - Re-run docs build locally to clear the `Build documentation` failures. - Make sure static checks pass locally: `prek run --from-ref main --stage pre-commit`. - Mark the PR as "Ready for review" when you're done. Converting a PR to draft is **not** a rejection — it is an invitation to bring the PR up to the project's standards so that maintainer review time is spent productively. There is no rush — take your time and work at your own pace. We appreciate your contribution and are happy to wait for updates. If you have questions, feel free to ask on the [Airflow Slack](https://s.apache.org/airflow-slack). -- 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]
