ipe: Add version 7.2.29#17890
Conversation
📝 WalkthroughWalkthroughThis PR updates Estimated code review effort: 1 (Trivial) | ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.
Inline comments:
In `@bucket/ipe.json`:
- Line 5: The manifest's license field is incorrect: update the "license" value
in bucket/ipe.json from "Freeware" to the SPDX identifier "GPL-3.0-or-later" to
reflect Ipe's upstream GPLv3-or-later license (with its special CGAL linking
exception noted in commit message or description if needed), and review whether
runtime dependencies should be declared by adding standard Scoop identifiers
(e.g., vcredist entries) to the "suggest" array in the same manifest; after
making changes, run the recommended local checks (.\\bin\\checkver.ps1 -App ipe
-f, .\\bin\\formatjson.ps1 -App ipe) and test installation (scoop install
bucket/ipe.json -a win64) before committing.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 0e3d2f23-726e-4e30-8653-4ad3d714a134
📒 Files selected for processing (1)
bucket/ipe.json
|
/verify |
|
All changes look good. Wait for review from human collaborators. ipe
|
Close #11882
<manifest-name[@version]|chore>: <general summary of the pull request>