Skip to content

Honor include_dev_requirements flag in Uv.install()#9

Merged
MrLYC merged 2 commits intodevin/1770481980-add-uv-supportfrom
copilot/sub-pr-5-again
Feb 7, 2026
Merged

Honor include_dev_requirements flag in Uv.install()#9
MrLYC merged 2 commits intodevin/1770481980-add-uv-supportfrom
copilot/sub-pr-5-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 7, 2026

The include_dev_requirements parameter in Uv.install() was accepted but ignored, making it impossible to control whether dev dependencies were installed.

Changes:

  • Pass --no-dev to uv sync when include_dev_requirements=False (default)
  • Added test_install_with_dev() to verify flag behavior

Behavior:

uv.install()                              # uv sync --no-dev (excludes dev)
uv.install(include_dev_requirements=True) # uv sync (includes dev)

This matches Poetry's semantic interface where the flag controls dev dependency installation.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: MrLYC <6391488+MrLYC@users.noreply.github.com>
Copilot AI changed the title [WIP] Update UV support implementation based on review feedback Honor include_dev_requirements flag in Uv.install() Feb 7, 2026
Copilot AI requested a review from MrLYC February 7, 2026 18:10
@MrLYC MrLYC marked this pull request as ready for review February 7, 2026 18:36
@MrLYC MrLYC merged commit 4192ebd into devin/1770481980-add-uv-support Feb 7, 2026
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