Skip to content

feat: use uv.lock deps for the dep graph instead - #4059

Draft
aignas wants to merge 1 commit into
bazel-contrib:mainfrom
aignas:aignas.chore.alias-src-targets
Draft

feat: use uv.lock deps for the dep graph instead#4059
aignas wants to merge 1 commit into
bazel-contrib:mainfrom
aignas:aignas.chore.alias-src-targets

Conversation

@aignas

@aignas aignas commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Summary:

  • Thread deps from uv.lock file into the hub_repository
  • Use them to define the dep graph
  • Alter the pkg_aliases and render_pkg_aliases to be able
    to use the dep graph information if it is provided.

TODO:

  • Split the whl_library_targets into separate files, I think they
    have stabilized enough.
  • Have a separate PR just for threading the dependencies to
    render_pkg_aliases. We probably need to handle various edge-cases
    where we have different package versions in the uv.lock file.
  • Have a separate PR to enable usage of the dependency graph. This
    is now probably doing a few shortcuts.
  • If we have uv.lock, define fewer wheel repos - only define the
    sources without passing the dep_template in there.
  • Test coverage for the above.

Creating the PR for discussion mainly. I am not sure when I'll get to do any of
the above, so if someone would like to start doing this along the similar lines
on how this PR has been designed, please feel free to take any part here.

Summary:
- Thread deps from uv.lock file into the hub_repository
- Use them to define the dep graph
- Alter the pkg_aliases and render_pkg_aliases to be able
  to use the dep graph information if it is provided.

TODO:
- [ ] Split the whl_library_targets into separate files, I think they
  have stabilized enough.
- [ ] Have a separate PR just for threading the dependencies to
  render_pkg_aliases. We probably need to handle various edge-cases
  where we have different package versions in the uv.lock file.
- [ ] Have a separate PR to enable usage of the dependency graph. This
  is now probably doing a few shortcuts.
- [ ] If we have uv.lock, define fewer wheel repos - only define the
  sources without passing the dep_template in there.
- [ ] Test coverage for the above.
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.

1 participant