Skip to content

Fix broken DagRunApi.md docs link (filename casing)#156

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-dagrunapi-doc-case
Jun 1, 2026
Merged

Fix broken DagRunApi.md docs link (filename casing)#156
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-dagrunapi-doc-case

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Jun 1, 2026

The top-level README.md links to docs/DagRunApi.md, but the file on disk is docs/DAGRunApi.md — so the link 404s on case-sensitive filesystems (Linux/CI), and the name doesn't match the openapi-generator canonical output (DagRunApi.md). The wrong casing is a leftover from the 2.x client era that survived regenerations because case-only renames are dropped on case-insensitive (macOS) filesystems, where the release client is generated.

This renames the file to the canonical DagRunApi.md and updates the two remaining references to the old path (airflow_client/.openapi-generator/FILES and the stale airflow_client/README.md).

Found while verifying release reproducibility for #155.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8)

The top-level README links to docs/DagRunApi.md but the file on disk was
docs/DAGRunApi.md, so the link 404s on case-sensitive filesystems and does
not match the openapi-generator canonical output. The wrong casing is a
2.x-era leftover that survived regenerations because case-only renames are
dropped on case-insensitive (macOS) filesystems where the client is built.

Rename to canonical DagRunApi.md and update the two remaining references to
the old path.
@potiuk potiuk force-pushed the fix-dagrunapi-doc-case branch from a5c0921 to 56f0f3a Compare June 1, 2026 21:20
@potiuk potiuk requested a review from jedcunningham June 1, 2026 21:23
@potiuk potiuk requested a review from kaxil June 1, 2026 21:24
@potiuk potiuk merged commit 718955e into apache:main Jun 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants