snap.md: Update warning with deprecation date #589
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
davidjharder
commented
Dec 20, 2024
davidjharder
commented
- Update the warning on the snap page to include a date for deprecation. I avoid a precise date because we cannot say exactly when the change will be synced to stable
- Change the wording slightly to my preferred style
EbonJaeger
previously requested changes
Dec 20, 2024
davidjharder
force-pushed
the
cric-crac-croc
branch
from
522f1d3 to
4447471
Compare
2 tasks
TraceyC77
approved these changes
Dec 20, 2024
TraceyC77
added a commit
to getsolus/packages
that referenced
this pull request
Dec 20, 2024
**Summary** - Add a sentence to the warning which runs for every snap, telling users that snaps will stop working soon. - Part of #325 - See also getsolus/help-center-docs#589 Sync Message: The warning that pops up whenever you run a snap has been changed to include the sentence **"Snaps will stop working in early January 2025."** This is part of a process we started back in July to remove snap support from Solus; we wrote about our reasons for this in a [blog post](https://getsol.us/2024/07/15/dropping-apparmor-kernel-patches/). The Help Center has [a page](https://help.getsol.us/docs/user/software/third-party/snap/) with more details about the warning and what you can do if you have snaps installed. Most users will not have snaps installed, as Solus has usually recommended that third-party apps be installed from the Third Party section of the Solus Software Center or as Flatpaks. **Test Plan** - Run a CLI snap, see changed warning - Graphical warnings have a bug and do not appear, separate from this change. **Checklist** - [x] Package was built and tested against unstable - [x] This change could gainfully be listed in the weekly sync notes once merged <!-- Write an appropriate message in the Summary section -->
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.