-
Notifications
You must be signed in to change notification settings - Fork 1.6k
locale-remulator: Add version 1.6.0 #16778
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Add new manifest for Locale Remulator.
WalkthroughAdds a new Scoop manifest Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (5 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
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 |
|
Your changes did not pass all checks. Please address the issues in the manifest and comment starting with locale-remulator
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
bucket/locale-remulator.json (1)
20-33: Write LRConfig.xml to$persist_dir(and create the directory) to match the guard + persist behavior.
Right now you check$persist_dir\LRConfig.xmlbut write to$dir\LRConfig.xml, which can become order-dependent across installs/upgrades."pre_install": [ "if (!(Test-Path \"$persist_dir\\LRConfig.xml\")) {", + " if (!(Test-Path \"$persist_dir\")) { New-Item -Path \"$persist_dir\" -ItemType Directory -Force | Out-Null }", " $content = @(", " '<LRConfig><Profiles Type=\"exe\">'", " '<Profile Name=\"Run in Japanese\" Guid=\"11111111-1111-1111-1111-111111111111\"><Location>ja-JP</Location><CodePage>932</CodePage><LCID>1041</LCID><TimeZone>Tokyo Standard Time</TimeZone><Bias>540</Bias><RunAsAdmin>false</RunAsAdmin><HookIME>false</HookIME><HookLCID>true</HookLCID></Profile>'", " '<Profile Name=\"Run in Japanese (Admin)\" Guid=\"22222222-2222-2222-2222-222222222222\"><Location>ja-JP</Location><CodePage>932</CodePage><LCID>1041</LCID><TimeZone>Tokyo Standard Time</TimeZone><Bias>540</Bias><RunAsAdmin>true</RunAsAdmin><HookIME>true</HookIME><HookLCID>true</HookLCID></Profile>'", " '<Profile Name=\"Run in Taiwan (Admin)\" Guid=\"33333333-3333-3333-3333-333333333333\"><Location>zh-TW</Location><CodePage>950</CodePage><LCID>1028</LCID><TimeZone>Taipei Standard Time</TimeZone><Bias>480</Bias><RunAsAdmin>true</RunAsAdmin><HookIME>true</HookIME><HookLCID>true</HookLCID></Profile>'", " '<Profile Name=\"Run in Korean (Admin)\" Guid=\"44444444-4444-4444-4444-444444444444\"><Location>ko-KR</Location><CodePage>949</CodePage><LCID>1042</LCID><TimeZone>Korea Standard Time</TimeZone><Bias>540</Bias><RunAsAdmin>true</RunAsAdmin><HookIME>true</HookIME><HookLCID>true</HookLCID></Profile>'", " '</Profiles></LRConfig>'", " )", - " Set-Content \"$dir\\LRConfig.xml\" -Value $content -Encoding Ascii", + " Set-Content \"$persist_dir\\LRConfig.xml\" -Value $content -Encoding Ascii", "}" ],
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
bucket/locale-remulator.json(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: o-l-a-v
Repo: ScoopInstaller/Extras PR: 16378
File: bucket/compactgui.json:20-22
Timestamp: 2025-10-19T13:58:23.389Z
Learning: In the ScoopInstaller/Extras repository, the CompactGUI manifest removed hash verification from the autoupdate block because the hash verification mechanism (scraping SHA-256 from release page HTML) is no longer available in newer CompactGUI releases. GitHub asset digests exist in beta releases but not in v3.8.0, and Scoop doesn't have built-in support for extracting from GitHub API asset digests.
Learnt from: Lutra-Fs
Repo: ScoopInstaller/Extras PR: 16020
File: bucket/regedix.json:6-8
Timestamp: 2025-09-26T02:55:31.132Z
Learning: For Scoop manifests, prefer using "suggest" over "depends" for .NET runtime dependencies because user machines often have externally managed .NET runtimes already installed, and using "depends" could cause duplicate installations of the same runtime.
📚 Learning: 2025-10-19T13:58:23.389Z
Learnt from: o-l-a-v
Repo: ScoopInstaller/Extras PR: 16378
File: bucket/compactgui.json:20-22
Timestamp: 2025-10-19T13:58:23.389Z
Learning: In the ScoopInstaller/Extras repository, the CompactGUI manifest removed hash verification from the autoupdate block because the hash verification mechanism (scraping SHA-256 from release page HTML) is no longer available in newer CompactGUI releases. GitHub asset digests exist in beta releases but not in v3.8.0, and Scoop doesn't have built-in support for extracting from GitHub API asset digests.
Applied to files:
bucket/locale-remulator.json
🪛 GitHub Actions: Pull Requests
bucket/locale-remulator.json
[error] 1-1: Autoupdate failed: Could not update locale-remulator, hash for Locale_Remulator_1.6.0.zip failed!
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: WindowsPowerShell
🔇 Additional comments (2)
bucket/locale-remulator.json (2)
10-12: Good call usingsuggestfor vcredist instead ofdepends(avoids forcing runtime installs).
Based on learnings, this matches common Extras repo guidance.
55-57: Thecheckver.githubformat is correct. Per the Scoop manifest schema, this field must be a full URI (e.g.,https://github.com/owner/repo), not the shorthandowner/repoformat. The code as shown is compliant.
|
/verify |
|
All changes look good. Wait for review from human collaborators. locale-remulator
|
locale-remulator: Add version 1.6.0
Closes #16726
Supersedes #11473
Added
locale-remulatoras requested. This is similar with Locale Emulator with x64 support.Verification:
install-context.ps1script.LRConfig.xml(auto-generates default config if missing).Manifest Changes:
Added
pre_installscript to generate a validLRConfig.xmlwith default profiles to preventLREditorcrash on first run.Added
post_installscript to facilitate context menu registration without requiring UAC during installation.Added
extract_dirto handle the nested directory structure.Use conventional PR title:
<manifest-name[@version]|chore>: <general summary of the pull request>I have read the Contributing Guide
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.