Skip to content

Implement --with-credentials flag in UV export_requirements#8

Merged
devin-ai-integration[bot] merged 3 commits intodevin/1770481980-add-uv-supportfrom
copilot/sub-pr-5
Feb 7, 2026
Merged

Implement --with-credentials flag in UV export_requirements#8
devin-ai-integration[bot] merged 3 commits intodevin/1770481980-add-uv-supportfrom
copilot/sub-pr-5

Conversation

Copy link
Contributor

Copilot AI commented Feb 7, 2026

The with_credentials parameter was accepted but silently ignored in UV's export_requirements method. This breaks PackageExtractor when using UV with private packages, since it calls export_requirements(..., with_credentials=True) expecting credentials to be embedded in the output.

Changes

  • versifier/uv.py: Pass --with-credentials flag to uv export when with_credentials=True
  • tests/test_uv.py: Assert flag presence/absence in both credential scenarios
# Before: credentials silently dropped
uv.export_requirements(with_credentials=True)  # Flag ignored

# After: credentials properly embedded
uv.export_requirements(with_credentials=True)  # Adds --with-credentials to uv export

Matches existing Poetry behavior where --with-credentials embeds authentication data in requirements.txt for private indices.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 7, 2026 18:09
Co-authored-by: MrLYC <6391488+MrLYC@users.noreply.github.com>
Co-authored-by: MrLYC <6391488+MrLYC@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix UV support implementation based on review feedback Implement --with-credentials flag in UV export_requirements Feb 7, 2026
Copilot AI requested a review from MrLYC February 7, 2026 18:11
@MrLYC MrLYC marked this pull request as ready for review February 7, 2026 18:37
@devin-ai-integration devin-ai-integration bot merged commit ef86fd4 into devin/1770481980-add-uv-support Feb 7, 2026
0 of 6 checks 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.

2 participants