Skip to content

Fix checkforupdates.sh: remove set -e and invalid snap refresh flag#11

Closed
Copilot wants to merge 2 commits intoUpdated-scripts-and-readme-2026-Febfrom
copilot/sub-pr-8-another-one
Closed

Fix checkforupdates.sh: remove set -e and invalid snap refresh flag#11
Copilot wants to merge 2 commits intoUpdated-scripts-and-readme-2026-Febfrom
copilot/sub-pr-8-another-one

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 25, 2026

set -e combined with an invalid snap refresh --stable flag caused the script to abort before running Flatpak updates or printing the completion message.

Changes

checkforupdates.sh

  • Removed set -e
  • Fixed snap refresh --stablesnap refresh (--stable is not a valid flag)
  • Wrapped Snap and Flatpak refresh in explicit error checks — failures emit a warning and execution continues
if ! snap refresh; then
    echo "Warning: Snap refresh encountered errors, continuing..."
fi

README.md

  • Updated checkforupdates feature description and troubleshooting section to reflect that Snap/Flatpak failures are non-fatal

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: andreak3779 <11012375+andreak3779@users.noreply.github.com>
Copilot AI changed the title [WIP] Update shell configuration instructions in README Fix checkforupdates.sh: remove set -e and invalid snap refresh flag Feb 25, 2026
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