ci: Remove pytest-localserver from requirements-testing.txt - #6612
Merged
alexander-alderman-webb merged 11 commits intoJun 24, 2026
Merged
ci: Remove pytest-localserver from requirements-testing.txt#6612alexander-alderman-webb merged 11 commits into
pytest-localserver from requirements-testing.txt#6612alexander-alderman-webb merged 11 commits into
Conversation
![sentry-warden[bot]](https://cdn.statically.io/img/avatars.githubusercontent.com/in/2755838?s=60&v=4){kind=small}
Contributor
Codecov Results 📊✅ 89835 passed | ⏭️ 6136 skipped | Total: 95971 | Pass Rate: 93.61% | Execution Time: 317m 40s 📊 Comparison with Base Branch
All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 2403 uncovered lines. Coverage diff@@ Coverage Diff @@
## main #PR +/-##
==========================================
- Coverage 89.90% 89.89% -0.01%
==========================================
Files 192 192 —
Lines 23763 23763 —
Branches 8206 8206 —
==========================================
+ Hits 21362 21360 -2
- Misses 2401 2403 +2
- Partials 1344 1346 +2Generated by Codecov Action |
alexander-alderman-webb
changed the title
alexander-alderman-webb
marked this pull request as ready for review
June 23, 2026 09:20
alexander-alderman-webb
deleted the
webb/requirements-testing/pytest-localserver
branch
June 24, 2026 07:53
6 tasks
mgaligniana
pushed a commit
to mgaligniana/sentry-python
that referenced
this pull request
Aug 9, 2026
…entry#6612) Remove the package from `requirements-testing.txt` and add it only as a dependency of the test suites that rely on it. Add a dependency on `werkzeug`, which was transitively pulled in by `pytest-localserver`. Add `dataclasses` as a dependency for `sanic` tests on Python 3.6, to avoid an unhandled `ImportError` that arose after moving the `pytest-localserver` dependency. This is part of environment cleanup before moving the remaining testing dependencies to a uv dependency group.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Remove the package from
requirements-testing.txtand 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
uv run ruff.feat:,fix:,ref:,meta:)