Skip to content

refactor!: Replace httpx with impit#560

Merged
vdusek merged 5 commits into
apify:masterfrom
Mantisus:httpx-to-impit
Aug 21, 2025
Merged

refactor!: Replace httpx with impit#560
vdusek merged 5 commits into
apify:masterfrom
Mantisus:httpx-to-impit

Conversation

@Mantisus

Copy link
Copy Markdown
Collaborator

Description

  • Replace the usage of HTTPX with Impit

Issues

@Mantisus
Mantisus temporarily deployed to fork-pr-integration-tests August 20, 2025 14:17 — with Image GitHub Actions Inactive
@Mantisus Mantisus self-assigned this Aug 20, 2025
@Mantisus
Mantisus requested review from Pijukatel and vdusek August 20, 2025 14:17
Comment thread pyproject.toml
@Mantisus
Mantisus temporarily deployed to fork-pr-integration-tests August 21, 2025 07:00 — with Image GitHub Actions Inactive
@vdusek
vdusek temporarily deployed to fork-pr-integration-tests August 21, 2025 08:01 — with Image GitHub Actions Inactive

@vdusek vdusek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

I just removed the httpx from the dev dependencies, we use the tool.mypy.overrides.ignore_missing_imports for such cases, to avoid bloating the dev dependencies.

@vdusek
vdusek merged commit cca3869 into apify:master Aug 21, 2025
19 of 21 checks passed
@vdusek vdusek added the t-tooling Issues with this label are in the ownership of the tooling team. label Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace the usage of HTTPX with Impit

3 participants