diff options
| author | Craig Jennings <c@cjennings.net> | 2026-01-24 13:33:06 -0600 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-01-24 13:33:06 -0600 |
| commit | 364fa17f9578b51f9a8e950684ee910e911ced59 (patch) | |
| tree | f32f67caeb353997ed2234b8a106eec31e1249e2 /todo.org | |
| parent | 6011f90328d88a2c449442d6a31fef3614926ec9 (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.org | 7 |
1 files changed, 7 insertions, 0 deletions
@@ -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. |
