Skip to content

Use uv --package instead of --project in codegen prek hooks - #71511

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-uv-package-codegen-hooks
Aug 12, 2026
Merged

Use uv --package instead of --project in codegen prek hooks#71511
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-uv-package-codegen-hooks

Conversation

@potiuk

@potiuk potiuk commented Aug 12, 2026

Copy link
Copy Markdown
Member

The generate-tasksdk-datamodels and generate-airflowctl-datamodels prek hooks
pass workspace package names (apache-airflow-task-sdk, apache-airflow-ctl) to
uv run --project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with:

error: Project directory `apache-airflow-task-sdk` does not exist

--package is the flag that takes a workspace member name. Verified against both
uv 0.11.29 and uv 0.12.3: both hooks pass and regenerate byte-identical output.

This currently blocks the CI environment upgrade, which bumps uv to 0.12.3.

Note that CI here cannot demonstrate the failure, since main still pins uv
0.11.29, where both spellings work. This PR shows no regression; the fix is
exercised by the upgrade PR.

related: #70501


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

Generated-by: Claude Code (Opus 5) following the guidelines

The two datamodel-codegen hooks passed workspace package names to
--project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with "Project directory does not exist",
blocking the CI environment upgrade. --package is the flag that takes a
member name, and it works on both versions.
@potiuk
potiuk merged commit c74cfcb into apache:main Aug 12, 2026
76 checks passed
@potiuk
potiuk deleted the fix-uv-package-codegen-hooks branch August 12, 2026 18:38
@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-3-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-3-test PR Link

github-actions Bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Aug 12, 2026
…ks (apache#71511)

The two datamodel-codegen hooks passed workspace package names to
--project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with "Project directory does not exist",
blocking the CI environment upgrade. --package is the flag that takes a
member name, and it works on both versions.
(cherry picked from commit c74cfcb)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Aug 12, 2026
…1511)

The two datamodel-codegen hooks passed workspace package names to
--project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with "Project directory does not exist",
blocking the CI environment upgrade. --package is the flag that takes a
member name, and it works on both versions.
aws-airflow-bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Aug 12, 2026
…ks (apache#71511)

The two datamodel-codegen hooks passed workspace package names to
--project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with "Project directory does not exist",
blocking the CI environment upgrade. --package is the flag that takes a
member name, and it works on both versions.
(cherry picked from commit c74cfcb)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
dabla pushed a commit to dabla/airflow that referenced this pull request Aug 14, 2026
…1511)

The two datamodel-codegen hooks passed workspace package names to
--project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with "Project directory does not exist",
blocking the CI environment upgrade. --package is the flag that takes a
member name, and it works on both versions.
potiuk added a commit that referenced this pull request Aug 17, 2026
…ks (#71511)

The two datamodel-codegen hooks passed workspace package names to
--project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with "Project directory does not exist",
blocking the CI environment upgrade. --package is the flag that takes a
member name, and it works on both versions.
(cherry picked from commit c74cfcb)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
potiuk added a commit that referenced this pull request Aug 17, 2026
…ks (#71511) (#71512)

The two datamodel-codegen hooks passed workspace package names to
--project, which takes a directory. uv 0.11 tolerated it; uv 0.12
validates the path and fails with "Project directory does not exist",
blocking the CI environment upgrade. --package is the flag that takes a
member name, and it works on both versions.
(cherry picked from commit c74cfcb)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants