Skip to content

Switch from httpx to httpxyz#924

Merged
teutoburg merged 3 commits into
mainfrom
fh/httpxyz
Jun 3, 2026
Merged

Switch from httpx to httpxyz#924
teutoburg merged 3 commits into
mainfrom
fh/httpxyz

Conversation

@teutoburg
Copy link
Copy Markdown
Contributor

@teutoburg teutoburg commented May 8, 2026

@hugobuddel and myself learned about this at PyGrunn today. I don't see a reason not to switch. Note that this does not yet remove httpx from our dependency tree, because skycalc-ipy still uses it. Will be switched in the next release there.

@teutoburg teutoburg self-assigned this May 8, 2026
@teutoburg teutoburg added the dependencies Related to or updating any dependencies label May 8, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.17%. Comparing base (55c06ee) to head (9f42f20).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #924   +/-   ##
=======================================
  Coverage   75.17%   75.17%           
=======================================
  Files          70       70           
  Lines        9024     9024           
=======================================
  Hits         6784     6784           
  Misses       2240     2240           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@teutoburg teutoburg marked this pull request as ready for review May 8, 2026 20:41
@teutoburg teutoburg requested a review from hugobuddel May 8, 2026 20:41
Copy link
Copy Markdown
Collaborator

@hugobuddel hugobuddel left a comment

Choose a reason for hiding this comment

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

Sorry for leaving you hanging here @teutoburg !

@teutoburg
Copy link
Copy Markdown
Contributor Author

Sorry for leaving you hanging here @teutoburg !

Nah, all good. I'm about to change my mind on this and go to niquests anyway, which I'm now using in a new project. But we can also do that later and merge this now. Well, after I fixed the lock file conflict that is...

@teutoburg teutoburg merged commit e79eec2 into main Jun 3, 2026
45 of 47 checks passed
@teutoburg teutoburg deleted the fh/httpxyz branch June 3, 2026 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Related to or updating any dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants