CI: Fix uv.lock by removing coordinator distribution#1585
Merged
jason810496 merged 2 commits intoMay 18, 2026
Merged
Conversation
uranusjr
approved these changes
May 18, 2026
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0f73cf062c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Member
Author
|
@codex review |
There was a problem hiding this comment.
Pull request overview
Cleanup PR following #1580 (which moved the coordinator implementation back into the SDK). Removes leftover references to the now-deleted apache-airflow-coordinators-java workspace member from the root uv.lock, pyproject.toml workspace members list, docker-compose test mount files, and the namespace-extending shim in task-sdk/src/airflow/sdk/__init__.py.
Changes:
- Remove
apache-airflow-coordinators-javafromuv.lockmembers and its package entry. - Remove
sdk/coordinators/javaworkspace member from rootpyproject.toml. - Remove coordinator mount entries from docker-compose tests/remove sources YAMLs and drop the
pkgutil.extend_pathshim inairflow.sdk.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| uv.lock | Drops the coordinator member and its package metadata block; incidental marker/wheel diffs from uv lock regeneration. |
| pyproject.toml | Removes sdk/coordinators/java from workspace members. |
| scripts/ci/docker-compose/tests-sources.yml | Removes coordinator tests bind mount. |
| scripts/ci/docker-compose/remove-sources.yml | Removes coordinator src empty mount. |
| task-sdk/src/airflow/sdk/init.py | Removes the pkgutil.extend_path namespace shim no longer needed after coordinator moved into SDK. |
|
Codex Review: Didn't find any major issues. Chef's kiss. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
jason810496
merged commit ebe061c
into
task-sdk/feature/coordinator-interface
4 checks passed
jason810496
deleted the
task-sdk/feature/coordinator-interface-fix-uv-lock
branch
jason810496
added a commit
that referenced
this pull request
May 22, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
jason810496
added a commit
that referenced
this pull request
May 23, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
jason810496
added a commit
that referenced
this pull request
May 23, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 25, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 26, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 26, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 26, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 27, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 27, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 27, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 27, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 27, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
uranusjr
pushed a commit
that referenced
this pull request
May 27, 2026
* CI: Fix uv.lock by removing coordinator distribution * Remove the remaining entry catch by codex bot
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
We should remove
apache-airflow-coordinators-javauv member from rootuv.lock.