Skip to content

backrest: Add restic dependency#17332

Merged
z-Fng merged 2 commits intoScoopInstaller:masterfrom
eawd123wadd4354as243df:backrest-add-restic-dependency
Mar 4, 2026
Merged

backrest: Add restic dependency#17332
z-Fng merged 2 commits intoScoopInstaller:masterfrom
eawd123wadd4354as243df:backrest-add-restic-dependency

Conversation

@eawd123wadd4354as243df
Copy link
Contributor

@eawd123wadd4354as243df eawd123wadd4354as243df commented Mar 3, 2026

On windows, restic is not installed by when running backrest for the first time.
Added the required dependency.

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

Summary by CodeRabbit

  • Chores
    • Added a runtime dependency declaration for restic in the package manifest, ensuring restic is installed and available at runtime. This change updates the package metadata to include restic as a required dependency.

added restic dependency
@coderabbitai
Copy link

coderabbitai bot commented Mar 3, 2026

Walkthrough

Added a top-level depends field with the string value "restic" to bucket/backrest.json, placed after the "license" field; one line added.

Changes

Cohort / File(s) Summary
Manifest Dependency Declaration
bucket/backrest.json
Inserted top-level depends field set to "restic" (one-line addition) in the package manifest.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

review-needed

Poem

🐰 A small new line I penned with care,
"restic" now waits within the lair.
A manifest tidy, neat and bright,
Hop we fixed it — snug and right.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description includes the required conventional PR title checkbox, confirms reading the Contributing Guide, and provides context about the restic dependency issue on Windows.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title 'backrest: Add restic dependency' directly and clearly describes the main change: adding restic as a dependency to backrest.

✏️ 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

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 3, 2026

Your changes did not pass all checks.

Please address the issues in the manifest and comment starting with /verify to rerun the checks.

backrest

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

Check the full log for details.

@z-Fng
Copy link
Member

z-Fng commented Mar 4, 2026

/verify

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

All changes look good.

Wait for review from human collaborators.

backrest

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

Check the full log for details.

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 changed the title backrest@1.12.0: Added restic dependency backrest: Added restic dependency Mar 4, 2026
@z-Fng z-Fng changed the title backrest: Added restic dependency backrest: Add restic dependency Mar 4, 2026
@z-Fng z-Fng merged commit ba4b9b8 into ScoopInstaller:master Mar 4, 2026
3 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