Skip to content

Conversation

desmondcheongzx
Copy link
Collaborator

@desmondcheongzx desmondcheongzx commented Aug 30, 2025

Changes Made

  1. Fix typo
  2. Add proper links in example

@github-actions github-actions bot added the chore label Aug 30, 2025
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Summary

This PR makes a minor correction to the Turbopuffer integration documentation. The change fixes a typo in the configuration example by updating the namespace value from 'daft-tpuf-example2' to 'daft-tpuf-example', removing what appears to be an unintended '2' suffix.

The modification is contained within the Turbopuffer connector documentation file (docs/connectors/turbopuffer.md) and specifically affects line 33 where the namespace configuration is demonstrated. This change standardizes the example naming convention and improves documentation consistency without impacting any functional code or behavior. The Turbopuffer connector allows users to write data from Daft DataFrames to Turbopuffer vector databases, and this documentation provides users with configuration examples for establishing connections.

Confidence score: 5/5

  • This PR is extremely safe to merge with no risk of production issues
  • Score reflects that this is purely a documentation typo fix with no functional impact
  • No files require special attention as this is a single-line documentation change

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

@desmondcheongzx desmondcheongzx merged commit 81c8a2d into main Aug 30, 2025
30 checks passed
@desmondcheongzx desmondcheongzx deleted the desmond/cleanup-turbopuffer-guide branch August 30, 2025 00:37
Jay-ju pushed a commit to Jay-ju/Daft that referenced this pull request Sep 1, 2025
## Changes Made

1. Fix typo
2. Add proper links in example
venkateshdb pushed a commit to venkateshdb/Daft that referenced this pull request Sep 6, 2025
## Changes Made

1. Fix typo
2. Add proper links in example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant