Skip to content

Exclusion presets: add Firefox Snap Cache#2086

Merged
m3nu merged 1 commit intoborgbase:masterfrom
Chaikney:master
Oct 18, 2024
Merged

Exclusion presets: add Firefox Snap Cache#2086
m3nu merged 1 commit intoborgbase:masterfrom
Chaikney:master

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