diff options
| author | Craig Jennings <c@cjennings.net> | 2026-04-19 06:54:01 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-04-19 06:54:01 -0500 |
| commit | 716506accd3897dd2480affa16abf3fe883f0c6e (patch) | |
| tree | b78b0c36a0f6bb9155fd4747c4a02c60c52f4fbf /modules | |
| parent | 1a8243e90499dab97463a783f76f10e40e7a38e4 (diff) | |
| download | dotemacs-716506accd3897dd2480affa16abf3fe883f0c6e.tar.gz dotemacs-716506accd3897dd2480affa16abf3fe883f0c6e.zip | |
fix(reconcile): restore repo iteration under projects-dir and code-dir
The outer dolist in cj/check-for-open-work guarded its body with
(boundp 'base-dir), which always returns nil under lexical-binding
because base-dir is a lexical loop variable. Every repo under
projects-dir and code-dir was silently skipped; only org-dir and
user-emacs-directory (both top-level defvars) still got reconciled.
Remove the bogus boundp check. Add regression tests covering the
entry point itself — the existing suite only exercised the helpers.
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/reconcile-open-repos.el | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/reconcile-open-repos.el b/modules/reconcile-open-repos.el index a7236754..87c16a31 100644 --- a/modules/reconcile-open-repos.el +++ b/modules/reconcile-open-repos.el @@ -102,7 +102,7 @@ Returns a list of directory paths that contain a .git subdirectory." ;; these are constants defined in init.el ;; recursively find and check all git repos under these directories (dolist (base-dir (list projects-dir code-dir)) - (when (and (boundp 'base-dir) base-dir (file-directory-p base-dir)) + (when (and base-dir (file-directory-p base-dir)) (dolist (repo (cj/find-git-repos base-dir)) (cj/reconcile-git-directory repo)))) |