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 | 151d49d01dfe441b59d6dab30033b02d5b13523c (patch) | |
| tree | 36449853cecc49d084b85a8ead6748b2ac11dac6 /modules | |
| parent | bdf3058542bbce298b435b301c7863b9d81d5bf4 (diff) | |
| download | dotemacs-151d49d01dfe441b59d6dab30033b02d5b13523c.tar.gz dotemacs-151d49d01dfe441b59d6dab30033b02d5b13523c.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)))) |