Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 23, 2025

The requests library was listed in requirements.txt but unused in the codebase.

Changes

  • Removed requests==2.32.5 from requirements.txt

The library currently has no HTTP client implementation. If one is needed in the future, httpx is recommended as a modern alternative with async support and HTTP/2 capabilities.

Original prompt

This section details on the original issue you should resolve

<issue_title>Switch pyopds2 HTTP client implementation from requests to httpx</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: mekarpeles <978325+mekarpeles@users.noreply.github.com>
@mekarpeles mekarpeles marked this pull request as ready for review November 23, 2025 19:11
Copilot AI changed the title [WIP] Switch pyopds2 HTTP client implementation to httpx Remove unused requests dependency Nov 23, 2025
@mekarpeles mekarpeles merged commit e7d9f67 into main Nov 23, 2025
4 checks passed
@mekarpeles mekarpeles deleted the copilot/switch-http-client-to-httpx branch November 23, 2025 19:12
Copilot AI requested a review from mekarpeles November 23, 2025 19:12
mekarpeles added a commit that referenced this pull request Nov 23, 2025
* Remove unused requests dependency from requirements.txt

Co-authored-by: Michael E. Karpeles <michael.karpeles@gmail.com>
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.

Switch pyopds2 HTTP client implementation from requests to httpx

2 participants