Skip to content

aegisub: Persist catalog folder#17576

Merged
z-Fng merged 1 commit into
ScoopInstaller:masterfrom
emotion3459:patch
Apr 9, 2026
Merged

aegisub: Persist catalog folder#17576
z-Fng merged 1 commit into
ScoopInstaller:masterfrom
emotion3459:patch

Conversation

@emotion3459

@emotion3459 emotion3459 commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Aegisub uses the catalog folder to store user-created style presets. The current manifest does not persist these, so this PR fixes that.

  • 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 configuration to persist additional catalog data alongside existing backup and settings data, ensuring this information is maintained across sessions.

@coderabbitai

coderabbitai Bot commented Apr 9, 2026

Copy link
Copy Markdown

Walkthrough

Updated the persist configuration array in bucket/aegisub.json to include the "catalog" path alongside existing entries. This is a straightforward configuration modification affecting the application's persistence behavior.

Changes

Cohort / File(s) Summary
Configuration Update
bucket/aegisub.json
Added "catalog" to the persist configuration array, extending the list of paths to be persisted.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

review-needed

Poem

🐰 A catalog joins the persisting throng,
Where config arrays march along,
One line added, clean and bright,
Aegisub's memory set just right! 📚✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The PR description includes a clear summary explaining why the change is needed (catalog folder stores user-created style presets) and includes both required checkboxes completed and confirmed by the author.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title 'aegisub: Persist catalog folder' accurately summarizes the main change: adding the catalog folder to the persist configuration in the aegisub manifest.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@emotion3459

Copy link
Copy Markdown
Contributor Author

/verify

@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

aegisub

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

Check the full log for details.

@z-Fng z-Fng changed the title aegisub: persist catalog folder aegisub: Persist catalog folder Apr 9, 2026

@z-Fng z-Fng left a comment

Copy link
Copy Markdown
Member

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 11c5bc4 into ScoopInstaller:master Apr 9, 2026
4 checks passed
@emotion3459 emotion3459 deleted the patch branch April 9, 2026 07:32
@coderabbitai coderabbitai Bot mentioned this pull request Apr 29, 2026
2 tasks
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