summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2025-12-01 06:13:35 -0600
committerCraig Jennings <c@cjennings.net>2025-12-01 06:13:35 -0600
commit03145a135d854f9aceb82ad65ce47a6a39235094 (patch)
tree01ceb7e83231f12b101f9e57507fca90496821f0
parentbcacfa0b3ae83ab6e509d798f8fcc2344a8c003d (diff)
docs(TODO): mark root check and git pull fixes as verified complete
Both fixes were already implemented: - Root check exists at archsetup:23-27 - git_install and yay installer use safe rm + fresh clone (no git pull --force) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
-rw-r--r--TODO.org13
1 files changed, 7 insertions, 6 deletions
diff --git a/TODO.org b/TODO.org
index 2fcea23..cf33cea 100644
--- a/TODO.org
+++ b/TODO.org
@@ -148,15 +148,16 @@ The script handles edge cases gracefully, provides detailed error messages with
CLOSED: [2025-11-13 Wed]
RESOLVED - VM test confirms dotfiles are properly stowed as symlinks; all configs and scripts in place
-*** TODO [#A] Add root check at script start
+*** DONE [#A] Add root check at script start
+CLOSED: [2025-12-01 Sun]
Prevents running with wrong permissions that cause silent failures
-Script will fail if not run as root but doesn't check until first privileged command
-Add at top of script: ~[ "$EUID" -ne 0 ] && echo "Must run as root" && exit 1~
+RESOLVED: Root check already exists at archsetup:23-27
-*** TODO [#A] Fix git pull --force data loss risk
+*** DONE [#A] Fix git pull --force data loss risk
+CLOSED: [2025-12-01 Sun]
Could destroy user's uncommitted changes in repos during dotfile setup
-Using ~git pull --force origin master~ can destroy local modifications without warning (archsetup:128)
-Should use ~git pull~ or ~git fetch && git reset --hard origin/master~ with user confirmation
+RESOLVED: Already fixed - git_install and yay installer use safe rm + fresh clone approach.
+No git pull --force in current script.
*** TODO [#A] Add pre-flight checks before installation starts
Validate system requirements: disk space, network connectivity, Arch version, tool dependencies (curl, stow, git, make)