Update Python Client to 3.2.2#155
Conversation
potiuk
left a comment
There was a problem hiding this comment.
Verified the generated client is reproducible from the committed spec/v2.yaml — nothing was hand-edited; the whole airflow_client/client/ and test/ trees regenerate byte-for-byte (after the trigger_dag_run_post_body AST patch under Python 3.10). LGTM.
One thing to flag for the toolchain (not blocking this PR): this was generated with openapi-generator-cli v7.22.0, but apache/airflow's breeze currently pins v7.21.0. With 7.21.0, three files diverge (api_client.py, configuration.py, connection_body.py — newer-generator codegen for sanitize_for_serialization, the retries type, and validator coercion). 3.2.1 matched 7.21.0, so the generator drifted between releases. I'm bumping breeze's pin to 7.22.0 so future regenerations stay self-consistent.
Minor, harmless: docs/DAGRunApi.md keeps its 3.2.1-era casing while the generator now emits DagRunApi.md (content identical) — a case-only rename dropped on a case-insensitive (macOS) filesystem. Worth a follow-up git mv so it matches canonical output, but it doesn't affect the importable package.
Drafted-by: Claude Code (Opus 4.8); reviewed by @potiuk before posting
|
Correction to my review above: I checked against a slightly stale Drafted-by: Claude Code (Opus 4.8); reviewed by @potiuk before posting |
No description provided.