Skip to content

thonny: Update to version 5.0.0, fix portable asset rename#18321

Open
l46983284-cpu wants to merge 1 commit into
ScoopInstaller:masterfrom
l46983284-cpu:thonny-5.0.0-asset-rename
Open

thonny: Update to version 5.0.0, fix portable asset rename#18321
l46983284-cpu wants to merge 1 commit into
ScoopInstaller:masterfrom
l46983284-cpu:thonny-5.0.0-asset-rename

Conversation

@l46983284-cpu

Copy link
Copy Markdown
Contributor

Closes #18318

Upstream 5.0.0 renamed the Windows portable zip to thonny-$version-windows-portable-x64.zip, so the old autoupdate pattern 404s. This PR bumps to 5.0.0, updates the hash from a full download of the new asset, and fixes checkver/autoupdate for the new name.

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

Upstream renamed the Windows portable zip from
thonny-$version-windows-portable.zip to
thonny-$version-windows-portable-x64.zip in 5.0.0.
Update current URL/hash and autoupdate/checkver so the
scheduled updater keeps working.

Signed-off-by: Alex Chen <l46983284@gmail.com>
@l46983284-cpu

Copy link
Copy Markdown
Contributor Author

/verify

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The Thonny bucket manifest is updated from version 4.1.7 to 5.0.0. The Windows portable download URL and hash are replaced, while the autoupdater regex now supports an optional -x64 suffix and targets the -windows-portable-x64.zip asset.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is specific, concise, and accurately reflects the version bump and portable asset rename.
Description check ✅ Passed The description includes the issue reference, change summary, and required checklist items from the template.
Linked Issues check ✅ Passed The changes update Thonny from 4.1.7 to 5.0.0 as requested in #18318.
Out of Scope Changes check ✅ Passed The PR stays focused on the manifest version bump, asset URL/hash update, and autoupdate pattern fix.
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

Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

thonny

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

Check the full log for details.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
bucket/thonny.json (1)

2-24: 📐 Maintainability & Code Quality | 🔵 Trivial

Run the standard Scoop validation before merging.

scoop config debug true
scoop config gh_token <your-github-token>
.\bin\checkver.ps1 -App thonny -f
.\bin\formatjson.ps1 -App thonny
scoop install bucket/thonny.json -a 64bit

As per path instructions, validate version detection, formatting, and installation locally.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@bucket/thonny.json` around lines 2 - 24, Run the standard Scoop validation
for the thonny manifest: enable debug mode and configure a GitHub token, then
execute checkver.ps1 with force, formatjson.ps1, and a 64-bit installation of
bucket/thonny.json. Resolve any version-detection, formatting, or installation
issues before merging.

Source: Path instructions

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@bucket/thonny.json`:
- Around line 2-24: Run the standard Scoop validation for the thonny manifest:
enable debug mode and configure a GitHub token, then execute checkver.ps1 with
force, formatjson.ps1, and a 64-bit installation of bucket/thonny.json. Resolve
any version-detection, formatting, or installation issues before merging.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: de9a2bc0-2269-4b20-9683-adeab0969b59

📥 Commits

Reviewing files that changed from the base of the PR and between 5866b27 and d3a5ae9.

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

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.

Update Thonny (thonny) to the latest version from 4.1.7 to 5.0.0

1 participant