llrt-beta: Add version 0.8.1-beta#2821
Conversation
WalkthroughA new package manifest for LLRT (Low Latency Runtime) beta version 0.8.1 is added, specifying metadata, Windows x64 artifact download link with hash verification, exposed executable, GitHub-based version checking, and autoupdate configuration. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested labels
Suggested reviewers
Poem
🚥 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)
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.
Pull request overview
Adds a new Scoop manifest for the LLRT beta release line, following the decision to publish it as llrt-beta (since upstream currently ships beta releases).
Changes:
- Introduce
bucket/llrt-beta.jsonmanifest pinned to LLRT0.8.1-beta - Configure GitHub-based
checkverandautoupdatefor future beta updates
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/verify |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
bucket/llrt-beta.json (1)
13-16: Consider restrictingcheckver.regexto beta tags for defensive consistency.While llrt currently uses the
-betasuffix consistently in all release tags (v0.8.1-beta, v0.8.0-beta, etc.) and no stable versions exist, the current regex/releases/tag/(?:v|V)?([\\d.\\w-]+)would technically match non-beta tags likev0.8.1if they were ever created. Tightening the regex to explicitly match only-betavariants makes the intent explicit and protects against future upstream tagging changes.Suggested change
"checkver": { "github": "https://github.com/awslabs/llrt", - "regex": "/releases/tag/(?:v|V)?([\\d.\\w-]+-beta[\\d.\\w-]*)" + "regex": "/releases/tag/(?:v|V)?([\\d.\\w-]+-beta[\\d.\\w-]*)" }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@bucket/llrt-beta.json` around lines 13 - 16, The checkver.regex in llrt-beta.json is too broad and may match non-beta tags; update the "checkver": "regex" to only accept versions ending with "-beta" (e.g., require the captured group to end with -beta) so it only matches release tags like v0.8.1-beta; modify the regex string under "checkver" (checkver.regex) accordingly to explicitly enforce the -beta suffix.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@bucket/llrt-beta.json`:
- Around line 13-16: The checkver.regex in llrt-beta.json is too broad and may
match non-beta tags; update the "checkver": "regex" to only accept versions
ending with "-beta" (e.g., require the captured group to end with -beta) so it
only matches release tags like v0.8.1-beta; modify the regex string under
"checkver" (checkver.regex) accordingly to explicitly enforce the -beta suffix.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 2d2061e8-b483-475b-aad5-69d312a7637a
📒 Files selected for processing (1)
bucket/llrt-beta.json
|
All changes look good. Wait for review from human collaborators. llrt-beta
|
This PR is a follow-up to ScoopInstaller/Extras#17672. Since LLRT currently provides beta releases, this manifest has been renamed from
llrttollrt-betaas suggested.Closes #2822
<manifest-name[@version]|chore>: <general summary of the pull request>Summary by CodeRabbit