kicad: Update to version 10.0.0-1, fix checkver & autoupdate#17508
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughUpdated Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 kicad
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@bucket/kicad.json`:
- Around line 8-9: The provided SHA-256 value is the empty-string placeholder
and must be replaced with the real hash for the file identified by the URL
"https://github.com/KiCad/kicad-source-mirror/releases/download/10.0.0/kicad-10.0.0-1-x86_64.exe#/dl.7z"
in kicad.json; download the exact artifact (kicad-10.0.0-1-x86_64.exe), compute
its SHA-256 digest using a trusted tool (e.g., sha256sum or equivalent), and
update the "hash" field in bucket/kicad.json with that computed hex digest to
replace the placeholder value.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
|
/verify |
|
Your changes did not pass all checks. Please address the issues in the manifest and comment starting with kicad
|
|
I added regex and changed autoupdate url to work version schemes: |
|
/verify |
|
All changes look good. Wait for review from human collaborators. kicad
|
|
|
/verify |
|
All changes look good. Wait for review from human collaborators. kicad
|
Previous file has been removed from github release page, new one has been added with
-1suffix<manifest-name[@version]|chore>: <general summary of the pull request>Summary by CodeRabbit