Skip to content

Conversation

@lvyuemeng
Copy link
Contributor

@lvyuemeng lvyuemeng commented Jan 13, 2026

Closes: #7528

  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

Summary by CodeRabbit

  • New Features
    • VMR is now available as a Windows package via Scoop, with support for 64-bit and ARM64 architectures and automatic updates.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 13, 2026

Walkthrough

A new Scoop package manifest is added for VMR, a cross-platform version manager tool. The manifest specifies installation details for Windows 64-bit and ARM64 architectures, including download URLs, SHA256 checksums, binary name, and automatic update configuration.

Changes

Cohort / File(s) Summary
Scoop Package Manifest
bucket/vmr.json
New manifest file defining VMR version 0.7.5 with architecture-specific downloads (64bit and arm64), checksums, executable binary mapping, GitHub version checking, and autoupdate configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A manifest appears with care,
For VMR, the version manager fair!
Windows paths now clearly laid,
Auto-updates, perfectly made! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'vmr: Add version 0.7.5' directly describes the main change: adding the vmr manifest for version 0.7.5, following the conventional format.
Description check ✅ Passed The PR description includes the issue reference (Closes: #7528) and confirms both checklist items required by the template (conventional title and contributing guide review).
Linked Issues check ✅ Passed The PR adds the vmr 0.7.5 manifest as requested in #7528, with appropriate metadata, download URLs, checksums, and autoupdate configuration for the package.
Out of Scope Changes check ✅ Passed All changes are scoped to adding the vmr manifest file; no unrelated modifications are present outside the linked issue requirements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eab15ab and 53c13fa.

📒 Files selected for processing (1)
  • bucket/vmr.json
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: narnaud
Repo: ScoopInstaller/Main PR: 7171
File: bucket/clink-flex-prompt.json:30-35
Timestamp: 2025-09-13T06:49:53.420Z
Learning: In Scoop package management, users typically upgrade packages rather than explicitly removing old versions. The `scoop cleanup` command and manual version removal are uncommon workflows, so optimizing for the primary upgrade scenario is reasonable.
🔇 Additional comments (1)
bucket/vmr.json (1)

1-30: Manifest structure is correct, but SHA256 hashes require manual verification.

The vmr package manifest follows standard Scoop conventions:

  • Proper metadata with version, description, homepage, and license
  • Architecture-specific download URLs for both 64bit (amd64) and arm64
  • Simplified GitHub checkver syntax
  • Autoupdate configuration with correct $version placeholder matching the v0.7.5 URL pattern

Both download URLs are accessible (HTTP 200 confirmed). However, the SHA256 hashes cannot be verified in the sandbox environment due to download timeouts. Manually verify the hashes match the actual release assets on the GitHub releases page:

  • 64bit: 8eee6c8b2e8af68184239cbc5fa3f2a49d1352b4e9e28a74206c5b374a42eef7
  • arm64: 3d9febab6a3bb68808f098a65fd26637d5b6fe402ece258d79fc124104c14725

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

Your changes did not pass all checks.

Please address the issues in the manifest and comment starting with /verify to rerun the checks.

vmr

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

@lvyuemeng
Copy link
Contributor Author

/verify

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

vmr

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

@z-Fng
Copy link
Member

z-Fng commented Jan 13, 2026

/verify

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

vmr

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

Copy link
Member

@z-Fng z-Fng left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

The manifest will update automatically as the package updates, as long as the upstream releases new packages that follow the same pattern defined in the manifest. If there are any other changes, feel free to submit a PR.

@z-Fng z-Fng merged commit 3c002d1 into ScoopInstaller:master Jan 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Request]: Vmr

2 participants