Skip to content

Update copy files CLI docs#4095

Merged
Wauplin merged 2 commits intomainfrom
copy-folder-is-allowed
Apr 13, 2026
Merged

Update copy files CLI docs#4095
Wauplin merged 2 commits intomainfrom
copy-folder-is-allowed

Conversation

@Wauplin
Copy link
Copy Markdown
Contributor

@Wauplin Wauplin commented Apr 13, 2026

Small doc update.

hf buckets cp is not only for files, works also on remote folders.

context (private DMs) https://huggingface.slack.com/archives/C07KX53FZTK/p1776090965972179

image

Note

Low Risk
Documentation/CLI help-text only changes; no functional behavior or security-sensitive logic is modified.

Overview
Updates hf buckets cp documentation and help text to better reflect supported sources/destinations (local files plus model/dataset hf://... handles, including folder-style paths for repo/bucket copies).

Also tweaks wording in the CLI guide to remove the “single files” framing and clarifies that bucket-to-repo copy is not yet supported.

Reviewed by Cursor Bugbot for commit ed82a16. Bugbot is set up for automated code reviews on this repo. Configure here.

@Wauplin Wauplin requested a review from julien-c April 13, 2026 14:43
Copy link
Copy Markdown
Member

@julien-c julien-c left a comment

Choose a reason for hiding this comment

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

thx!

@Wauplin Wauplin merged commit 182360b into main Apr 13, 2026
19 of 21 checks passed
@Wauplin Wauplin deleted the copy-folder-is-allowed branch April 13, 2026 14:45
@bot-ci-comment
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

* `SRC`: Source: local file, HF handle (hf://...), or - for stdin [required]
* `[DST]`: Destination: local path, HF handle (hf://...), or - for stdout
* `SRC`: Source: local file, model or dataset handle (hf://...), or - for stdin [required]
* `[DST]`: Destination: local path, model or dataset handle (hf://...), or - for stdout
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

uuh i read too fast, the DST should be bucket handle (and the source should be any handle)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

arf thanks yes. I should stop editing code manually 😃
opened #4096 to fix it

Comment on lines +940 to +942
src: Annotated[str, typer.Argument(help="Source: local file, model or dataset handle (hf://...), or - for stdin")],
dst: Annotated[
str | None, typer.Argument(help="Destination: local path, HF handle (hf://...), or - for stdout")
str | None, typer.Argument(help="Destination: local path, model or dataset handle (hf://...), or - for stdout")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

and same here, sorry

@Wauplin Wauplin mentioned this pull request Apr 13, 2026
@huggingface-hub-bot
Copy link
Copy Markdown
Contributor

This PR has been shipped as part of the v1.11.0 release.

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.

3 participants