Skip to content

rssguard@5.0.2: Update persist folder name#17339

Merged
z-Fng merged 1 commit intoScoopInstaller:masterfrom
desk7:rssguard-fix-persist-folder
Mar 4, 2026
Merged

rssguard@5.0.2: Update persist folder name#17339
z-Fng merged 1 commit intoScoopInstaller:masterfrom
desk7:rssguard-fix-persist-folder

Conversation

@desk7
Copy link
Contributor

@desk7 desk7 commented Mar 4, 2026

According to the docs, the data folder in portable mode now is called data5, after the release of version 5.0.0.

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

Summary by CodeRabbit

  • Chores
    • Updated internal configuration settings.

@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

Walkthrough

A configuration value in bucket/rssguard.json was updated, changing the persist property from "data4" to "data5".

Changes

Cohort / File(s) Summary
Configuration Update
bucket/rssguard.json
Updated persist property from "data4" to "data5"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny tweak in the JSON so neat,
Data4 bows, now Data5 takes the seat,
One simple change, the config's replete,
Persistence persists, ever so sweet! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description provides the rationale for the change, references official documentation, confirms reading the contributing guide, and uses the conventional PR title format as required.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and specifically describes the main change: updating the persist folder name for rssguard from data4 to data5 to match documentation.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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 and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

All changes look good.

Wait for review from human collaborators.

rssguard

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

Check the full log for details.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

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/rssguard.json`:
- Line 22: Add a "notes" array to the RSS Guard manifest next to the existing
"persist": "data5" entry documenting the incompatible database change from 4.x
("data4") to 5.x ("data5") and providing a migration path; follow the Scoop
bucket convention used in manifests like youtube-music.json and yapi.json and
include two lines: one stating that "RSS Guard 5.x uses 'data5' and is
incompatible with 4.x databases in 'data4'", and another instructing users
upgrading from 4.x to either import feeds via RSS Guard's built-in import
feature or manually copy 'data4' to 'data5' before first launch.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d09aa797-fa33-4bb3-bd2e-7b3a57cf03a2

📥 Commits

Reviewing files that changed from the base of the PR and between c18a09f and 374465d.

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

@z-Fng z-Fng changed the title rssguard@5.0.2: update persist folder name to match documentation rssguard@5.0.2: Update persist folder name Mar 4, 2026
Copy link
Member

@z-Fng z-Fng left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@z-Fng z-Fng merged commit 979c145 into ScoopInstaller:master Mar 4, 2026
4 checks passed
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.

2 participants