seanmuth commented on PR #69489: URL: https://github.com/apache/airflow/pull/69489#issuecomment-4906438281
Heads-up on CI: the only red check is `CI image checks / Static checks`, and it's **unrelated to this PR**. The failing hook is `update-providers-build-files`, which regenerates provider packaging files (`apache/kafka`, `apache/flink`, `github`, `zendesk`, `microsoft/mssql`, …) — this branch touches **zero** provider files (it's scoped to `airflow-core` assets API + a migration). It's pre-existing drift on `main` that any `airflow-core` PR triggering the `--all-files` static run inherits. Everything substantive is green: the asset filter tests, mypy, ruff, migration/revision-heads checks, and the full test matrix all pass (138 passing). I also validated the change end-to-end on a local 3.2.1 env and a real Astro deployment (Runtime 3.2-4): exact `?uri=` returns the single matching asset, repeated `?uri=a&uri=b` returns both, a partial value correctly returns nothing, and `uri_pattern` still does its substring match. This is ready for another look. Thanks again for the review! --- Drafted-by: Claude Code (Opus 4.8, 1M context) (no human review 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]
