Fix style inconsistencies in bin/ (#7518)

* Use (( )) for the numeric argument test

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Drop the quotes on a variable inside [[ ]]

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Use omarchy-pkg-drop instead of raw pacman -Rns

omarchy-pkg-drop already filters to installed packages, so the
2>/dev/null || true suppression is no longer needed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Drop defensive checks around default-set commands

ttfx, imagemagick, and networkmanager are all in the default package
set, so their commands are runtime invariants and should be invoked
directly. Removing the nmcli guard also removes the degraded wifi
fallthrough that only ran when nmcli was missing.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
David Heinemeier Hansson
2026-08-19 20:16:47 +02:00
committed by GitHub
co-authored by Claude Fable 5
parent 3451719ca5
commit 9455496990
6 changed files with 12 additions and 26 deletions
+2 -2
View File
@@ -10,7 +10,7 @@ if [[ -z $1 ]]; then
fi
remove_php() {
sudo pacman -Rns --noconfirm php composer php-sqlite xdebug 2>/dev/null || true
omarchy-pkg-drop php composer php-sqlite xdebug
}
case "$1" in
@@ -50,7 +50,7 @@ laravel)
;;
symfony)
echo -e "Removing Symfony CLI...\n"
sudo pacman -Rns --noconfirm symfony-cli 2>/dev/null || true
omarchy-pkg-drop symfony-cli
;;
python)
echo -e "Removing Python...\n"