Skip to content

chore: Update HF docs to include more clarity around datasets and revisions.#345

Open
nicholasmccrea wants to merge 2 commits intomainfrom
no-ticket/update-hf-docs
Open

chore: Update HF docs to include more clarity around datasets and revisions.#345
nicholasmccrea wants to merge 2 commits intomainfrom
no-ticket/update-hf-docs

Conversation

@nicholasmccrea
Copy link
Contributor

@nicholasmccrea nicholasmccrea commented Feb 19, 2026

  • Add more clarity around datasets and how to download them
  • Add more clarity around revision
  • Use example model and dataset names instead of real models/datasets
  • Added HF to available upstreams matrix

Copilot AI review requested due to automatic review settings February 19, 2026 12:08
@vercel
Copy link

vercel bot commented Feb 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cloudsmith-docs Ready Ready Preview, Comment Feb 19, 2026 3:02pm

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Hugging Face repository documentation to use non-realistic example names and to better explain downloading datasets and selecting a revision for downloads.

Changes:

  • Replaced real-world model references with example names in Python and CLI snippets.
  • Added a Python example for downloading datasets via repo_type="dataset".
  • Added revision explanation notes for both huggingface_hub and hf CLI download flows.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +203 to +208
The `revision` parameter defaults to `"main"` if omitted. The value can be:

- A **branch name**, e.g., `revision="main"` or `revision="dev"`
- A **full commit SHA**, e.g., `revision="a1b2c3d4e5f67890abcdef1234567890abcdef12"`
- A **version tag**, e.g., `revision="v1.0"` or `revision="1.2.3"`

Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

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

In this Revision note, calling revision a "version tag" is ambiguous given the earlier section where a Hugging Face tag maps to a Cloudsmith package tag. Consider renaming this bullet to "tag name" or "git tag" to keep terminology consistent and avoid implying Cloudsmith package versions are semver strings.

Copilot uses AI. Check for mistakes.

- A **branch name**, e.g., `--revision main` or `--revision dev`
- A **full commit SHA**, e.g., `--revision a1b2c3d4e5f67890abcdef1234567890abcdef12`
- A **version tag**, e.g., `--revision v1.0` or `--revision 1.2.3`
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

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

Same terminology issue here: describing --revision v1.0 / --revision 1.2.3 as a "version tag" can be confusing given the page's tag/version mapping. Prefer "tag name" / "git tag" to align with the earlier Tags section.

Suggested change
- A **version tag**, e.g., `--revision v1.0` or `--revision 1.2.3`
- A **tag name**, e.g., `--revision v1.0` or `--revision 1.2.3`

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments