Skip to content

ci: Pin test dependencies including package extras#6607

Open
alexander-alderman-webb wants to merge 2 commits into
masterfrom
webb/toxgen/include-extras
Open

ci: Pin test dependencies including package extras#6607
alexander-alderman-webb wants to merge 2 commits into
masterfrom
webb/toxgen/include-extras

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Description

Ensure that any additional packages installed with extras are also pinned in our test matrix.

Some packages are installed with extras based on the package entry in config.py.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review June 19, 2026 07:23
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner June 19, 2026 07:23
Comment thread scripts/populate_tox/populate_tox.py Outdated
@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

91980 passed | ⏭️ 6140 skipped | Total: 98120 | Pass Rate: 93.74% | Execution Time: 321m 3s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +1181
Passed Tests 📈 +1170
Failed Tests
Skipped Tests 📈 +11

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2411 uncovered lines.
✅ Project coverage is 89.84%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.84%    89.84%        —%
==========================================
  Files          192       192         —
  Lines        23740     23740         —
  Branches      8194      8194         —
==========================================
+ Hits         21329     21329         —
- Misses        2411      2411         —
- Partials      1343      1344        +1

Generated by Codecov Action

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.

4 participants