Skip to content

tinyrdm: Add version 1.2.7#18262

Open
arvdk wants to merge 1 commit into
ScoopInstaller:masterfrom
arvdk:tinyrdm
Open

tinyrdm: Add version 1.2.7#18262
arvdk wants to merge 1 commit into
ScoopInstaller:masterfrom
arvdk:tinyrdm

Conversation

@arvdk

@arvdk arvdk commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Closes #17425

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

@arvdk

arvdk commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

/verify

@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 67199d5a-4585-418a-8c2b-cf9916ba2d79

📥 Commits

Reviewing files that changed from the base of the PR and between f92a400 and 91b23fb.

📒 Files selected for processing (1)
  • bucket/tinyrdm.json

📝 Walkthrough

Walkthrough

This PR adds a new Scoop bucket manifest file, bucket/tinyrdm.json, for TinyRDM version 1.2.7. The manifest includes description, homepage, and license metadata, a WebView2 dependency suggestion, executable and shortcut definitions, checkver configuration pointing to the GitHub repository, and architecture-specific (x64/arm64) download URLs with SHA256 hashes for both the current release and autoupdate templates.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely states the manifest update for TinyRDM version 1.2.7.
Description check ✅ Passed The description includes the linked issue, conventional title confirmation, and contributor guide acknowledgment.
Linked Issues check ✅ Passed The new manifest matches the request by packaging Tiny RDM with versioned downloads, homepage, and standard install metadata.
Out of Scope Changes check ✅ Passed The changes are limited to the TinyRDM Scoop manifest and stay within the requested packaging update scope.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

tinyrdm

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

Check the full log for details.

@arvdk

arvdk commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

I'm not sure if the suggest is of any benefit. In a Win11 sandbox, it does not want to launch, even after I Scooped the webview2. It only works when I accept to install the webview2 through tinyrdm itself.

After a quick search, I found a wails_tools.nsh in the tinyrdm repo, that maybe explains why:

SetRegView 64
	# If the admin key exists and is not empty then webview2 is already installed
	ReadRegStr $0 HKLM "SOFTWARE\WOW6432Node\Microsoft\EdgeUpdate\Clients\{F3017226-FE2A-4295-8BDF-00C3A9A7E4C5}" "pv"
    ${If} $0 != ""
        Goto ok
    ${EndIf}

    ${If} ${REQUEST_EXECUTION_LEVEL} == "user"
        # If the installer is run in user level, check the user specific key exists and is not empty then webview2 is already installed
	    ReadRegStr $0 HKCU "Software\Microsoft\EdgeUpdate\Clients{F3017226-FE2A-4295-8BDF-00C3A9A7E4C5}" "pv"
        ${If} $0 != ""
            Goto ok
        ${EndIf}
     ${EndIf}

(Edit: I see there is already a PR #17938 for a fix, so then the scooped webview2 would work just fine.)

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]: Tiny RDM

1 participant