git-xet: Add version 0.2.1#7563
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
I think adding a note to guide users is sufficient, since git xet install can be applied to different scopes. PS> git xet install --help
Perform the following action to register Git-Xet:
Set up the "xet" transfer agent under the name "lfs.customtransfer.xet" in
global Git config with the below values
path = git-xet
args = transfer
concurrent = <according to value from option "--concurrency" (default true)>
If "--concurrency" is specified, set "lfs.concurrenttransfers" in the global
Git config to this number.
Usage: git-xet.exe install [OPTIONS]
Options:
--system
Set the "xet" transfer agent and the number of concurrent LFS transfers in the system git config, e.g. /etc/gitconfig instead of the global git config (~/.gitconfig)
--local
Set the "xet" transfer agent and the number of concurrent LFS transfers in the local repository's git config, instead of the global git config (~/.gitconfig)
--path <PATH>
The path to the repository. This argument only applies when "--local" is specified
--concurrency <CONCURRENCY>
The number of concurrent LFS uploads/downloads. Default 8
-h, --help
Print help (see a summary with '-h')
-V, --version
Print version |
Thank you! That makes sense to me. I added the following note: |
|
/verify @coderabbitai full review |
|
All changes look good. Wait for review from human collaborators. git-xet
|
|
✅ Actions performedFull review triggered. |
📝 WalkthroughWalkthroughA new manifest file for git-xet (Xet protocol Git extension) has been added to the bucket directory with version 0.2.1, including metadata, architecture-specific download details, binary information, and autoupdate configuration. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@bucket/git-xet.json`:
- Line 4: Replace the broken homepage URL value for the "homepage" JSON key
(currently "https://huggingface.co/docs/hub/xet/") with the working URL
"https://huggingface.co/docs/hub/xet/index" in the JSON object (update the value
for the homepage property in the same entry).
Co-authored-by: z-Fng <54583083+z-Fng@users.noreply.github.com>
|
I believe the current
One possible solution are provided below: {
"checkver": {
"url": "https://api.github.com/repositories/855423236/releases?per_page=45",
"jsonpath": "$[?(@.prerelease == false)].assets[?(@.name =~ /git-xet/i)].browser_download_url",
"regex": "(?i)download/git-xet-v([\\d.]+)/git-xet"
}
} |
I'd suggest using repository names instead of repo IDs in the GitHub API, as the API now properly supports redirects. To complement this, we're going to introduce an automation to automatically correct and sync repo names in our buckets.
It should be fine for now. But if you'd like to make it more robust, feel free to submit a PR. |
Thank you for your explanation.
I will submit a PR later to refine it. |
git-xet: Add version v0.2.0
Closes: #7516
<manifest-name[@version]|chore>: <general summary of the pull request>Summary by CodeRabbit