Skip to content

Fix v0.2.4 Winget checksum#7

Merged
loreste merged 1 commit into
loreste:mainfrom
attahn:fix/winget-v0.2.4-checksum
Jul 19, 2026
Merged

Fix v0.2.4 Winget checksum#7
loreste merged 1 commit into
loreste:mainfrom
attahn:fix/winget-v0.2.4-checksum

Conversation

@attahn

@attahn attahn commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Updates the v0.2.4 Winget installer checksum to match the published Windows release asset. The manifest retained the v0.2.3 checksum, causing Package Seeds validation to fail.

Tests: bash scripts/publish-winget-seed.sh; Package Seeds passed.

Summary by CodeRabbit

  • Chores
    • Updated the Windows package installer manifest for loreste.mako version 0.2.4 by refreshing the SHA-256 checksum used to validate the installer, keeping all other metadata unchanged.

@attahn

attahn commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

@coderabbitai[bot] review

@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bd4fcec9-a0a2-48bf-8aa8-19fa26ba204f

📥 Commits

Reviewing files that changed from the base of the PR and between 11ca8d4 and 5e8d54a.

📒 Files selected for processing (1)
  • packaging/winget/loreste.mako.installer.yaml

📝 Walkthrough

Walkthrough

The Winget installer manifest for loreste.mako version 0.2.4 updates its InstallerSha256 value while preserving all other manifest fields.

Changes

Installer manifest

Layer / File(s) Summary
Update installer checksum
packaging/winget/loreste.mako.installer.yaml
Replaces the installer package SHA-256 checksum with the new value; the installer URL and nested file definitions remain unchanged.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: updating the Winget checksum for v0.2.4.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

@attahn: I’ll review the changes in #7.

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@attahn

attahn commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

Package Seeds passes on the focused branch. The remaining upstream CI failures match the current baseline and are addressed by #6 (network timeout, sanitizer/runtime UB, cross-target POSIX declarations, and the claims gate). The checksum commit integrated after #6 passes all 11 CI jobs: https://github.com/attahn/mako/actions/runs/29692077948

@attahn
attahn force-pushed the fix/winget-v0.2.4-checksum branch from 11ca8d4 to 5e8d54a Compare July 19, 2026 17:06
@loreste
loreste marked this pull request as ready for review July 19, 2026 17:08
@loreste
loreste merged commit 70a857f into loreste:main Jul 19, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants