summaryrefslogtreecommitdiff
path: root/todo.org
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-01-24 13:33:06 -0600
committerCraig Jennings <c@cjennings.net>2026-01-24 13:33:06 -0600
commit364fa17f9578b51f9a8e950684ee910e911ced59 (patch)
treef32f67caeb353997ed2234b8a106eec31e1249e2 /todo.org
parent6011f90328d88a2c449442d6a31fef3614926ec9 (diff)
fix(archsetup): code cleanup and style fixes
- Remove commented-out reflector code (timer handles updates) - Fix arithmetic expressions (remove unnecessary inner $) - Simplify file truncation (: > instead of cat /dev/null >) - Fix Xorg heredoc indentation - Add shellcheck bulk cleanup as priority D task
Diffstat (limited to 'todo.org')
-rw-r--r--todo.org7
1 files changed, 7 insertions, 0 deletions
diff --git a/todo.org b/todo.org
index 8a7cd76..5f5f849 100644
--- a/todo.org
+++ b/todo.org
@@ -655,6 +655,13 @@ Steps:
Reference: Removed from archsetup on 2025-11-15
+*** TODO [#D] Bulk shellcheck cleanup
+Reviewed 2026-01-24: ~128 warnings, mostly acceptable patterns or low-priority style issues.
+- SC2024 (sudo redirects) - acceptable, script runs as root
+- SC2174 (mkdir -p -m) - reviewed, not a practical issue
+- Various quoting warnings - high-priority ones already fixed
+Run =shellcheck archsetup= periodically to check for new issues, but bulk cleanup is low priority.
+
** Method 6: Add Wayland/Hyprland Desktop Alternative
Currently archsetup installs only Xorg with DWM. This method adds a Wayland-based alternative using Hyprland that preserves the same workflow and keybindings while modernizing the graphics stack.