Skip to content

Use dependency groups for Python dev deps#2038

Open
brettcannon wants to merge 1 commit into
github:mainfrom
brettcannon:brettcannon-update-python-extras
Open

Use dependency groups for Python dev deps#2038
brettcannon wants to merge 1 commit into
github:mainfrom
brettcannon:brettcannon-update-python-extras

Conversation

@brettcannon

Copy link
Copy Markdown
Contributor

This change moves the Python development dependencies out of the project optional-dependencies section and into a PEP 735 dependency group in python/pyproject.toml.

The repo's Python install guidance now uses uv pip install -e . --group dev, so the same dev setup is available through the newer dependency-group mechanism. I also validated the updated install command with uv.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 0430f7db-6da1-47a9-9a35-fec3f1bbc17e
Copilot AI review requested due to automatic review settings July 20, 2026 23:20
@brettcannon
brettcannon requested a review from a team as a code owner July 20, 2026 23:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Moves Python development dependencies to a PEP 735 dependency group and updates installation guidance accordingly.

Changes:

  • Preserves telemetry as an optional package extra.
  • Moves development tools into the dev dependency group.
  • Updates developer install commands to use --group dev.
Show a summary per file
File Description
python/pyproject.toml Defines the new development dependency group.
justfile Updates automated Python dependency installation.
CONTRIBUTING.md Updates contributor setup instructions.
.github/copilot-instructions.md Updates Python workflow guidance.

Review details

  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Medium

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