Skip to content

ci: Remove pytest-forked from requirements-testing.txt - #6611

Merged
alexander-alderman-webb merged 6 commits into
masterfrom
webb/requirements-testing/pytest-forked
Jun 26, 2026
Merged

ci: Remove pytest-forked from requirements-testing.txt#6611
alexander-alderman-webb merged 6 commits into
masterfrom
webb/requirements-testing/pytest-forked

Conversation

@alexander-alderman-webb

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

Copy link
Copy Markdown
Contributor

Description

Remove the package from requirements-testing.txt and add the package as a dependency to only the test suites that rely on the package (if applicable).

This is part of environment cleanup before moving the remaining testing dependencies to a uv dependency group.

Issues

Reminders

Comment thread scripts/populate_tox/config.py
@alexander-alderman-webb alexander-alderman-webb changed the title ci: Remove pytest-forked from requirements-testing.txt Jun 22, 2026
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

90246 passed | ⏭️ 6300 skipped | Total: 96546 | Pass Rate: 93.47% | Execution Time: 317m 5s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.91%    89.94%    +0.03%
==========================================
  Files          192       192         —
  Lines        23809     23809         —
  Branches      8218      8218         —
==========================================
+ Hits         21405     21413        +8
- Misses        2404      2396        -8
- Partials      1339      1337        -2

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review June 23, 2026 09:22
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner June 23, 2026 09:22
Comment thread scripts/populate_tox/tox.jinja

@ericapisani ericapisani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@alexander-alderman-webb
alexander-alderman-webb merged commit 44be095 into master Jun 26, 2026
270 of 272 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/requirements-testing/pytest-forked branch June 26, 2026 12:42
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
…#6611)

Remove the package from `requirements-testing.txt` and add it only as a dependency of the test suites that rely on it.
This is part of environment cleanup before moving the remaining testing dependencies to a uv dependency group.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants