Skip to content

Integrate httpcore2 documentation into the httpx2 docs site#997

Closed
Kludex wants to merge 1 commit into
mainfrom
integrate-httpcore2-docs
Closed

Integrate httpcore2 documentation into the httpx2 docs site#997
Kludex wants to merge 1 commit into
mainfrom
integrate-httpcore2-docs

Conversation

@Kludex
Copy link
Copy Markdown
Member

@Kludex Kludex commented May 31, 2026

The httpcore2 docs lived in a standalone src/httpcore2/mkdocs.yml that was never wired into the build and still pointed at the old encode/httpcore upstream. This merges them into the main docs site so they ship at httpx2.pydantic.dev alongside the httpx2 docs.

  • Move src/httpcore2/docs/*.md to docs/httpcore2/, rewriting httpcore -> httpcore2 (prose, import, pip install, mkdocstrings ::: directives) and repointing badge/PyPI/GitHub URLs at pydantic/httpx2.
  • Add an httpcore2 section to the root mkdocs.yml nav.
  • Delete the standalone src/httpcore2/mkdocs.yml and the now-empty src/httpcore2/docs/.

Verified with zensical build - all httpcore2 pages render and the ::: httpcore2.X API references resolve. The remaining build warnings are pre-existing and unrelated to httpcore2.

AI Disclaimer

This PR was developed with the assistance of either Claude or Codex. I've reviewed and verified the changes.

Move the httpcore2 docs into the main mkdocs build under an httpcore2
nav section, rewriting httpcore references to httpcore2 and repointing
encode URLs at pydantic/httpx2. Drop the standalone httpcore2 mkdocs
config, which was unbuilt and still pointed at the old upstream.
@github-actions
Copy link
Copy Markdown

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 31, 2026

Merging this PR will not alter performance

✅ 15 untouched benchmarks
⏩ 7 skipped benchmarks1


Comparing integrate-httpcore2-docs (17a4ebe) with main (999da32)

Open in CodSpeed

Footnotes

  1. 7 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 17a4ebef15

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/httpcore2/proxies.md
pool = httpcore.ConnectionPool(proxy=proxy)
proxy = httpcore2.Proxy("http://127.0.0.1:8080/")
pool = httpcore2.ConnectionPool(proxy=proxy)
r = proxy.request("GET", "https://www.example.com/")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Send proxied requests through the pool

In this first proxy example, copying the documented code will fail because httpcore2.Proxy is configuration for ConnectionPool and does not implement .request(). The pool created on the previous line is the object that should send the request, so this should call pool.request(...) or users following the new docs hit an AttributeError before making any proxied request.

Useful? React with 👍 / 👎.

@Kludex Kludex closed this May 31, 2026
@Kludex Kludex deleted the integrate-httpcore2-docs branch May 31, 2026 18:17
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.

1 participant