Skip to content

[core] Refactor spacemacs/write-custom-settings-to-dotfile#17219

Merged
bcc32 merged 6 commits intosyl20bnr:developfrom
bcc32:refactor-write-custom
Feb 20, 2026
Merged

[core] Refactor spacemacs/write-custom-settings-to-dotfile#17219
bcc32 merged 6 commits intosyl20bnr:developfrom
bcc32:refactor-write-custom

Conversation

@bcc32
Copy link
Collaborator

@bcc32 bcc32 commented Jan 31, 2026

Primarily motivated by fixing a small behavior bug in this function,
where saving the custom settings (such as by custom-save-all)
killed any buffer visiting the dotspacemacs file. This is annoying
when the user is separately visiting that file to edit their configs,
and simply unnecessary.

The remainder of the commits are various cleanups and improvements to
make the code more idiomatic and performant.

A user might be visiting it, in which case this behavior is quite
annoying, and in any case this is simply not necessary.
`enable-command` does not kill the buffer, as an example of
convention.
Even if the user has disabled auto indentation of Elisp files on save.
... instead of going through a separate string copy.
@bcc32 bcc32 force-pushed the refactor-write-custom branch from 969378f to d5b453f Compare February 20, 2026 20:50
@bcc32
Copy link
Collaborator Author

bcc32 commented Feb 20, 2026

Seeing no objections, I'm going to merge this.

@bcc32 bcc32 merged commit 8de1122 into syl20bnr:develop Feb 20, 2026
6 of 7 checks passed
@bcc32 bcc32 deleted the refactor-write-custom branch February 20, 2026 20:50
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.

1 participant