diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-03 21:34:05 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-03 21:34:05 -0500 |
| commit | 56039342a1edae50c9eab4eaaa3b4eaa2370ea28 (patch) | |
| tree | c46518a67f777cf6a5385d79756dd878708a4350 /tests/test-custom-comments-comment-reformat.el | |
| parent | 550c776159e52817efd8369c5de0d4448907dc1c (diff) | |
| download | dotemacs-56039342a1edae50c9eab4eaaa3b4eaa2370ea28.tar.gz dotemacs-56039342a1edae50c9eab4eaaa3b4eaa2370ea28.zip | |
refactor: drop dead intermediate C-s binding in selection-framework
`selection-framework.el` had two `keymap-global-set "C-s"` calls at module load. The first bound `C-s` to `consult-line`, then a later block rebound the same key to `cj/consult-line-or-repeat`. The second binding always won, so the first was dead configuration and made the file harder to reason about.
I removed the intermediate `consult-line` binding. The final `cj/consult-line-or-repeat` binding stays. Behavior is unchanged.
I added `tests/test-selection-framework-keybindings.el` with one smoke test: load the module with `use-package`, `consult-line`, and `vertico-repeat` stubbed, then assert `C-s` resolves to `cj/consult-line-or-repeat`. That locks in the cleanup so a future re-add of the dead binding would fail the test.
Diffstat (limited to 'tests/test-custom-comments-comment-reformat.el')
0 files changed, 0 insertions, 0 deletions