Skip to content
This repository was archived by the owner on Oct 10, 2025. It is now read-only.

Conversation

@sdht0
Copy link
Contributor

@sdht0 sdht0 commented Jul 4, 2025

Went through as many pages as I could and heavily refactored the docs to read well and have a consistent format.

Mainly the intro, getting started, and extensions sections.

It'd be good to have a second set of eyes to verify all these changes. Sorry for dumping these all in the same PR.

Changed the parameters docs from tables to lists everywhere.

Fixes #535

@sdht0 sdht0 requested review from prrao87 and ray6080 July 4, 2025 01:06
@sdht0 sdht0 changed the title Refactor the extensions docs Refactor the docs Jul 4, 2025
Copy link
Contributor

@andyfengHKU andyfengHKU left a comment

Choose a reason for hiding this comment

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

LGTM.

@acquamarin check DuckDB, Postgres, ... & FTS
@ray6080 check vector
@mewim check installation

@sdht0 sdht0 force-pushed the refactor-extensions branch from de4d917 to 385d09b Compare July 4, 2025 12:23
@sdht0 sdht0 force-pushed the refactor-extensions branch from 385d09b to aaa065c Compare July 4, 2025 12:28
@sdht0 sdht0 changed the title Refactor the docs Refactor the extension docs Jul 4, 2025
@sdht0
Copy link
Contributor Author

sdht0 commented Jul 4, 2025

Split out the non-extension changes to #545

@prrao87
Copy link
Contributor

prrao87 commented Jul 4, 2025

@sdht0 there's a broken link, pls fix (per the failing link check).

@sdht0 sdht0 requested a review from acquamarin July 4, 2025 13:32
Copy link
Contributor

@ray6080 ray6080 left a comment

Choose a reason for hiding this comment

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

Gone over changes on vector extension. Looks good. Thanks!

@sdht0 sdht0 merged commit d1ea68c into dev Jul 4, 2025
1 check passed
@sdht0 sdht0 deleted the refactor-extensions branch July 4, 2025 14:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants