Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclusion presets: add Firefox Snap Cache #2086

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Conversation

Chaikney
Copy link

@Chaikney Chaikney commented Oct 2, 2024

Description

I have taken the exclusions from the existing firefox set, and matched them with those files that exist in the Snap package. Based on the Snap in KDE Neon.
Only one file modified.

Related Issue

(#2085)

Motivation and Context

The existing presets to exclude firefox cache files cover flatpaks but not Snaps. I added them manually to my own backup exclusions but I think they are probably more widely applicable.

How Has This Been Tested?

Not in this format; the exclusions are those I use already and the formatting seems to be simple enough to not be very concerning. (Famous last words?)

Screenshots (if appropriate):

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have read the CONTRIBUTING guide.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

I provide my contribution under the terms of the license of this repository and I affirm the Developer Certificate of Origin.

I have taken the exclusions from the existing firefox set,
and matched them with those files that exist in the Snap pacakge.
Based on the Snap in KDE Neon.
@m3nu m3nu merged commit 0d3d644 into borgbase:master Oct 18, 2024
@Chaikney
Copy link
Author

Possibly my first "real" OSS contribution 🍾 World domination awaits!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants