Skip to content

keepass@2.61.1: Switch to portable releases - #18526

Closed
Vixb1122 wants to merge 1 commit into
ScoopInstaller:masterfrom
Vixb1122:keepass
Closed

keepass@2.61.1: Switch to portable releases#18526
Vixb1122 wants to merge 1 commit into
ScoopInstaller:masterfrom
Vixb1122:keepass

Conversation

@Vixb1122

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

@coderabbitai

coderabbitai Bot commented Aug 14, 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 Plus

Run ID: 85084f8a-7d5c-4298-b309-5b7f419b16b6

📥 Commits

Reviewing files that changed from the base of the PR and between 8e813d6 and 7eadd31.

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

📝 Walkthrough

Walkthrough

KeePass now downloads the 2.61.1 ZIP archive instead of the installer executable. The manifest uses the updated SHA-1 hash and no longer sets innosetup. Automatic updates now target the versioned ZIP archive URL.

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

Merge Risk: 🔵 Low · up to 7eadd

The manifest changes the KeePass artifact to a ZIP and updates its digest; an incorrect archive layout or hash could cause installation failures or fetch the wrong artifact. Merge is appropriate with owner awareness and completion of the requested validation checks.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description contains only the template comments and checkboxes, with no issue reference or actual change summary. Add a concise change summary and a valid Closes or Relates issue reference.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title follows the conventional format and clearly describes the KeePass change to portable releases.

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.

@z-Fng z-Fng changed the title keepass: Update url keepass@2.61.1: Update download & autoupdate URLs Aug 14, 2026
@z-Fng z-Fng added the blocked Being blocked by another PR or issue label Aug 14, 2026
@z-Fng z-Fng changed the title keepass@2.61.1: Update download & autoupdate URLs keepass@2.61.1: Switch to portable releases Aug 14, 2026
Comment thread bucket/keepass.json
},
"autoupdate": {
"url": "https://downloads.sourceforge.net/project/keepass/KeePass%20$majorVersion.x/$version/KeePass-$version-Setup.exe"
"url": "https://downloads.sourceforge.net/project/keepass/KeePass%20$majorVersion.x/$version/KeePass-$version.zip"

@z-Fng z-Fng Aug 14, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It would be helpful to clearly explain why this change is necessary. This ensures that you've searched the existing PRs and understand the implications of the change. This is a breaking change and duplicates #15024. See:

@z-Fng z-Fng added duplicate and removed blocked Being blocked by another PR or issue labels Aug 14, 2026
@z-Fng z-Fng closed this Aug 14, 2026
@Vixb1122
Vixb1122 deleted the keepass branch August 15, 2026 01:11
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.

2 participants