Skip to content

Conversation

bcc32
Copy link
Collaborator

@bcc32 bcc32 commented Sep 19, 2025

De facto, the values for the various dotspacemacs-* variables
default to whatever is in dotspacemacs-template.el. Make the actual
default values (if the variables are not explicitly set by
.spacemacs) match the user-facing defaults.

In this PR, I also:

  1. Fixed the tests to avoid loading the user's actual ~/.spacemacs,
    so that their behavior does not depend on the running user.
  2. Added a test to keep the values core-dotspacemacs.el and
    dotspacemacs-template.el in sync. (This test starts out failing
    until the commit which fixes core-dotspacemacs.el)

@bcc32 bcc32 marked this pull request as draft September 19, 2025 17:34
Rename test dotspacemacs.el to init.el, and use the test directory as
SPACEMACSDIR (which will cause Spacemacs to automatically load
$TEST_DIR/init.el as the dotspacemacs file and avoid reading the
"real" ~/.spacemacs).

Furthermore, use emacs -Q to avoid loading any site-lisp or site-start
files.

This change allows developers to easily run the tests and have correct
behavior without going through the GitHub CI scripts (instead just
running make locally).
De facto, the values for the various dotspacemacs-* variables default
to whatever is in dotspacemacs-template.el.  Make the actual default
values (if the variables are not explicitly set by .spacemacs) match
the user-facing defaults.
@bcc32 bcc32 marked this pull request as ready for review September 19, 2025 18:43
@fnussbaum fnussbaum merged commit 3df12f6 into syl20bnr:develop Sep 20, 2025
9 checks passed
@bcc32 bcc32 deleted the match-defaults branch September 22, 2025 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants