rescuetime: Update to version 3.2.12.3, enable checkver#18238
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe pull request updates the RescueTime manifest in Estimated code review effort: 1 (Trivial) | ~3 minutes Changes
Related issues: None provided. Related PRs: None provided. Suggested labels: manifest-update Suggested reviewers: None provided. 🚥 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/rescuetime.json`:
- Around line 20-23: The manifest has a checkver block for RescueTime, but it is
missing the matching autoupdate block needed for automated updates. Add an
autoupdate section that parameterizes the versioned top-level url used by the
RescueTime manifest so checkver -u can update version, url, and hash together;
keep the existing checkver definition and wire its template values to the same
version placeholder used by the manifest fields.
🪄 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: 500a9119-46c8-4ad2-88f9-e4dd2c1f2eca
📒 Files selected for processing (1)
bucket/rescuetime.json
caf17b0 to
7520c6c
Compare
|
/verify |
This comment was marked as outdated.
This comment was marked as outdated.
Refactor RescueTime JSON structure to include architecture details.
This comment was marked as outdated.
This comment was marked as outdated.
|
/verify |
|
All changes look good. Wait for review from human collaborators. rescuetime
|
z-Fng
left a comment
There was a problem hiding this comment.
Thank you for your contribution!
The manifest was stuck on 3.0.0.10 because there is no
checkver, so excavator never picked up new releases. This bumps it to the current 3.2.12.3 (hash computed from the official installer) and adds acheckveragainst the official release notes page so it stays up to date going forward.<manifest-name[@version]|chore>: <general summary of the pull request>