Skip to content

docs(model-providers): copy-editing across provider pages - #499

Merged
g-despot merged 2 commits into
docs/review-integrationfrom
docs/low-model-providers-copyedit
Aug 8, 2026
Merged

docs(model-providers): copy-editing across provider pages#499
g-despot merged 2 commits into
docs/review-integrationfrom
docs/low-model-providers-copyedit

Conversation

@g-despot

@g-despot g-despot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Low tier, 2 of 7. 4 findings, 18 files.

A duplicated-word footer ("multimodal embedding embeddings models") appears in eight provider pages, not the two filed. Plus a duplicated verb, a repeated Dataricks typo, and Chose for Choose across three Transformers pages.

Beyond the brief: the Databricks pages told readers to provide an API key in several places. Databricks uses a personal access token, which those pages' own header tables already say. The multimodal custom-image page also told readers to replace a folder name the Dockerfile beside it does not use.

Verified: build exit 0.

A duplicated-word footer ('multimodal embedding embeddings models') appears in
eight provider pages, a duplicated verb, a repeated 'Dataricks' typo, and
'Chose' for 'Choose' across three Transformers pages.

The Databricks pages told readers to provide an API key in several places.
Databricks uses a personal access token, which the pages' own header tables
already say. The custom-image page also named a folder the Dockerfile beside it
does not use.

@orca-security-eu orca-security-eu Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copy-edits 18 model-provider pages to improve grammar, terminology, and configuration accuracy.

Changes:

  • Removes duplicated words and corrects grammar and spelling.
  • Clarifies Databricks token requirements and Cohere defaults.
  • Aligns custom-image instructions with Dockerfile paths.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
voyageai/reranker.md Fixes duplicated embedding wording.
voyageai/embeddings.md Fixes duplicated embedding wording.
transformers/reranker.md Corrects “choose” grammar.
transformers/embeddings.md Corrects “choose” grammar.
transformers/embeddings-multimodal.md Corrects “choose” grammar.
transformers/embeddings-multimodal-custom-image.md Clarifies custom model paths and terminology.
transformers/embeddings-custom-image.md Corrects verb agreement.
nvidia/reranker.md Fixes duplicated embedding wording.
nvidia/generative.md Fixes duplicated embedding wording.
nvidia/embeddings.md Fixes duplicated embedding wording.
kubeai/index.md Removes a duplicated verb.
google/index.md Improves grammar and heading hierarchy.
databricks/generative.md Corrects spelling and token terminology.
databricks/embeddings.md Corrects spelling, spacing, and token terminology.
cohere/reranker.md Clarifies the server default and fixes duplication.
cohere/generative.md Fixes duplicated embedding wording.
cohere/embeddings.md Clarifies the server default and fixes duplication.
cohere/embeddings-multimodal.md Clarifies the server default.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/weaviate/model-providers/transformers/embeddings-multimodal-custom-image.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@g-despot
g-despot merged commit 54c2939 into docs/review-integration Aug 8, 2026
2 of 3 checks passed
@g-despot
g-despot deleted the docs/low-model-providers-copyedit branch August 8, 2026 14:04
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.

2 participants