Skip to content

docs: replace premature Ollama troubleshooting with embedding provider reference#223

Open
TerminalGravity wants to merge 1 commit intomainfrom
fix/remove-premature-ollama-docs
Open

docs: replace premature Ollama troubleshooting with embedding provider reference#223
TerminalGravity wants to merge 1 commit intomainfrom
fix/remove-premature-ollama-docs

Conversation

@TerminalGravity
Copy link
Collaborator

The README troubleshooting section included an 'Ollama embeddings connection refused' entry, but Ollama isn't supported on main — it's only in feature branches (feat/ollama-embeddings, etc.). This confused users into thinking Ollama was a valid provider option.

Replaced with a clear table showing the two supported providers (local, openai) and a note that Ollama/Voyage are coming soon.

…r reference

The Ollama troubleshooting section referenced a provider that doesn't
exist on main yet (only in feature branches). Replaced with a clear
table of supported providers (local, openai) and a note that Ollama
and Voyage support is coming soon.
Copy link
Collaborator Author

@TerminalGravity TerminalGravity left a comment

Choose a reason for hiding this comment

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

Good call removing the premature Ollama docs — documenting features that don't exist yet confuses users. The replacement table is clean.

Minor: with #222 adding Voyage support, the "Coming soon" note will be partially stale on merge. Might want to coordinate — either merge #222 first and update this PR, or land this first and have #222 update the table.

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