docs: port clickhouse-c language client docs from upstream#163
Open
mintlify[bot] wants to merge 1 commit into
Open
docs: port clickhouse-c language client docs from upstream#163mintlify[bot] wants to merge 1 commit into
mintlify[bot] wants to merge 1 commit into
Conversation
1 task
Contributor
Author
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Ports the new
clickhouse-c(header-only C native protocol client) docs from clickhouse/clickhouse-docs commit91c27a1into the Mintlify site.Changes
integrations/language-clients/c.mdxmirroring upstreamdocs/integrations/language-clients/c.md(slug/integrations/cpreserved).sidebar_label→sidebarTitle, droppedsidebar_position) and:::noteadmonition →<Note>.integrations/language-clients/navigation.json.Context
Notes / ambiguities
/images/integrations/logos/logo_cpp.svg) for the nav icon — there is no dedicated C logo asset in this repo yet. Worth swapping if a proper C icon is added.cpp.mdxpage uses slug/interfaces/cppwhile its upstream uses/integrations/cpp. Per instructions I kept the upstream slug/integrations/cfor this new page; if the Mintlify convention should follow the C++ deviation (/interfaces/c), let me know.mint broken-linksflags/interfaces/tcp,/interfaces/formats/Native,/interfaces/http,/cloud/overview. These targets all exist (e.g.concepts/features/interfaces/tcp.mdxdeclares slug/interfaces/tcp); the warnings are pre-existing checker noise, not regressions from this change.Note
Low Risk
Documentation-only addition with no application or runtime code changes.
Overview
Adds ClickHouse C client (
clickhouse-c) documentation to the Mintlify site by porting upstream reference content into a new page at/integrations/c.The new
integrations/language-clients/c.mdxcovers the header-only native-protocol client end to end: deliberate non-goals, header layout, vendoring withCHC_IMPLEMENTATION, TCP connect/query/insert flows, column layouts, compression, OpenSSL I/O, and the ioless async client. Mintlify-specific tweaks includesidebarTitle, a<Note>admonition, and preserved anchor IDs.integrations/language-clients/navigation.jsongains a C sidebar group (rootintegrations/language-clients/c) placed after C++; the nav icon temporarily reuses the C++ logo asset.Reviewed by Cursor Bugbot for commit 2c920fe. Bugbot is set up for automated code reviews on this repo. Configure here.