Skip to content

Conversation

@rafaelha
Copy link
Contributor

@rafaelha rafaelha commented Nov 3, 2025

This ensures uv sync installs all dependencies required for the unit tests.

It's not ideal because some dependencies are now duplicated, but I'm not aware of a method to reference optional dependencies (extras) in uv groups.

@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
9893 8697 88% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: de63585 by action🐍

Comment on lines +77 to +83
extras = [
"lark>=1.2.2",
"cirq-core>=1.4.1",
"cirq-core[contrib]>=1.4.1",
"qpsolvers[clarabel]>=4.7.0",
"stim>=1.15.0",
]
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. also maybe avoid extras as key because uv use --extras as flag
  2. you can actually do this:
extras = [
    "bloqade-circuit[qasm, stim, cirq]"
]
dev = [
     {include-group = "extras"},
    "black>=24.10.0",
    "coverage>=7.6.4",
    "ipython>=8.29.0",
    "isort>=5.13.2",
    "mypy>=1.13.0",
    "numpy>=1.26.4",
    "pre-commit>=4.0.1",
    "pyright>=1.1.388",
    "pytest>=8.3.3",
    "ruff>=0.7.3",
    "rust-just>=1.36.0",
    "tomlkit>=0.13.2",
]

Then this is also pip compatible (your solution here with default group only work for uv

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.

3 participants