ci: Remove httpcore from requirements-testing.txt - #6622
ci: Remove httpcore from requirements-testing.txt#6622alexander-alderman-webb merged 15 commits into
Conversation
Sorry, something went wrong.
Codecov Results 📊✅ 89837 passed | ⏭️ 6240 skipped | Total: 96077 | Pass Rate: 93.51% | Execution Time: 327m 30s 📊 Comparison with Base Branch
All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 2405 uncovered lines. @@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 89.86% 89.88% +0.02%
==========================================
Files 192 192 —
Lines 23763 23763 —
Branches 8206 8206 —
==========================================
+ Hits 21353 21358 +5
- Misses 2410 2405 -5
- Partials 1346 1345 -1Generated by Codecov Action |
Sorry, something went wrong.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 12da4cd. Configure here.
Sorry, something went wrong.
ericapisani
left a comment
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
Remove the package from `requirements-testing.txt` and add it only as a dependency of the test suites that rely on it. Add `dataclasses` as a dependency for all tests, to avoid unhandled `ImportError` exceptions that arose after moving the `httpcore` dependency. This is part of environment cleanup before moving the remaining testing dependencies to a uv dependency group.
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