chore: Remove the AlwaysShowNews option (#288)

The AlwaysShowNews was only introduced to keep compatibility with the historic & initial Arch-Update's behavior of always showing news on upgrade (regardless of whether there's a new one since the last run or not). I think it's fair to say that this was a poor design that just created undesired annoyances to people. As such, I'm not sure the AlwaysShowNews option is really desired / used by anyone and therefore could be deprecated & removed altogether. Closes #284 --------- Co-authored-by: Robin Candau <[email protected]>
Antiz96 · Dec 6, 2024 · 8f5fdee · 8f5fdee
1 parent 7ea5325
commit 8f5fdee
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 15 deletions.
diff --git a/README-fr.md b/README-fr.md
@@ -237,7 +237,6 @@ Les options prises en charge sont :
 
 - NoColor # Ne pas coloriser la sortie.
 - NoVersion # Ne pas afficher les modifications de versions des paquets lors du listing des mises à jour en attente (y compris lors de l'utilisation de l'option `-l / --list`).
-- AlwaysShowNews # Toujours afficher les Arch news avant de mettre à jour, peu importe s'il y en a une nouvelle depuis la dernière exécution ou non.
 - NewsNum=[Num] # Nombre de Arch news à affcher avant la mise à jour et avec l'option `-n / --news` (voir la page de manuel arch-update(1) pour plus de details). La valeur par défaut est 5.
 - AURHelper=[AUR Helper] # AUR helper à utiliser pour la prise en charge des paquets AUR. Les valeurs valides sont `paru`, `yay` ou `pikaur`. Si cette option n'est pas spécifiée, Arch-Update utilisera le premier AUR helper disponible dans l'ordre suivant : `paru` puis `yay` puis `pikaur` (si aucun d'eux n'est installé, Arch-Update ne prendra pas en compte les paquets AUR).
 - PrivilegeElevationCommand=[Cmd] # Commande à utiliser pour l'élévation de privilège. Les valeurs valides sont `sudo`, `doas` ou `run0`. Si cette option n'est pas spécifiée, Arch-Update utilisera la première commande disponible dans l'odre suivant : `sudo`, `doas` puis `run0`.

diff --git a/README.md b/README.md
@@ -237,7 +237,6 @@ The supported options are:
 
 - NoColor # Do not colorize output.
 - NoVersion # Do not show versions changes for packages when listing pending updates (including when using the `-l / --list` option).
-- AlwaysShowNews # Always display Arch news before updating, regardless of whether there's a new one since the last run or not.
 - NewsNum=[Num] # Number of Arch news to display before updating and with the `-n / --news` option (see the arch-update(1) man page for more details). Defaults to 5.
 - AURHelper=[AUR Helper] # AUR helper to be used for AUR packages support. Valid values are `paru`, `yay` or `pikaur`. If this option is not set, Arch-Update will use the first available AUR helper in the following order: `paru` then `yay` then `pikaur` (in case none of them is installed, Arch-Update will not take AUR packages into account).
 - PrivilegeElevationCommand=[Cmd] # Command to be used for privilege elevation. Valid values are `sudo`, `doas` or `run0`. If this option is not set, Arch-Update will use the first available command in the following order: `sudo`, `doas` then `run0`.

diff --git a/doc/man/arch-update.conf.5 b/doc/man/arch-update.conf.5
@@ -27,10 +27,6 @@ Do not colorize output.
 .B NoVersion
 .RB "Do not show versions changes for packages when listing pending updates (including when using the " "-l / --list " "option, see the " "arch-update(1) " "man page for more details)."
 
-.TP
-.B AlwaysShowNews
-Always display Arch news before updating, regardless of whether there's a new one since the last run or not.
-
 .TP
 .B NewsNum=[Num]
 .RB "Number of Arch news to display before updating and with the " "-n / --news " "option (see the " "arch-update(1) " "man page for more details). Defaults to 5."

diff --git a/doc/man/fr/arch-update.conf.5 b/doc/man/fr/arch-update.conf.5
@@ -27,10 +27,6 @@ Ne pas coloriser la sortie.
 .B NoVersion
 .RB "Ne pas afficher les modifications de versions des paquets lors du listing des mises à jour en attente (y compris lors de l'utilisation de l'option " "-l / --list" ", voir la page de man " "arch-update(1) " "pour plus de détails)."
 
-.TP
-.B AlwaysShowNews
-Toujours afficher les Arch news avant de mettre à jour, peu importe s'il y en a une nouvelle depuis la dernière exécution ou non.
-
 .TP
 .B NewsNum=[Num]
 .RB "Nombre de Arch news à afficher avant la mise à jour et avec l'option " "-n / --news " "(voir la page de man " "arch-update(1) " "pour plus de détails). La valeur par défaut est 5."

diff --git a/res/config/arch-update.conf.example b/res/config/arch-update.conf.example
@@ -4,7 +4,6 @@
 
 #NoColor
 #NoVersion
-#AlwaysShowNews
 #NewsNum=5
 #AURHelper=paru
 #PrivilegeElevationCommand=sudo

diff --git a/src/lib/config.sh b/src/lib/config.sh
@@ -18,10 +18,6 @@ if [ -f "${config_file}" ]; then
 	# shellcheck disable=SC2034
 	no_version=$(grep -Eq '^[[:space:]]*NoVersion[[:space:]]*$' "${config_file}" 2> /dev/null && echo "true")
 
-	# Check the "AlwaysShowNews" option in arch-update.conf
-	# shellcheck disable=SC2034
-	show_news=$(grep -Eq '^[[:space:]]*AlwaysShowNews[[:space:]]*$' "${config_file}" 2> /dev/null && echo "true")
-
 	# Check the "NewsNum" option in arch-update.conf
 	# shellcheck disable=SC2034
 	news_num=$(grep -E '^[[:space:]]*NewsNum[[:space:]]*=[[:space:]]*[1-9][0-9]*[[:space:]]*$' "${config_file}" 2> /dev/null | awk -F '=' '{print $2}' | tr -d '[:space:]')