diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-24 09:02:29 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-24 09:02:29 -0500 |
| commit | 1b86dd5b738d9b0b272f0d01b8913b2ae8938e78 (patch) | |
| tree | 20c8237215339ffe0ca63131fb85088285eef296 /modules/keybindings.el | |
| parent | d9ab1074991ee3c4c680024b14d6de0475560380 (diff) | |
| download | dotemacs-1b86dd5b738d9b0b272f0d01b8913b2ae8938e78.tar.gz dotemacs-1b86dd5b738d9b0b272f0d01b8913b2ae8938e78.zip | |
fix(githooks): stop the secret scan passing when it read nothing
- The scan piped git diff into grep and swallowed failures with || true.
- grep exits 1 on no matches, which is normal, so the guard had to stay.
- But it also hid a git failure, leaving nothing to scan and reporting clean.
- Now the diff is read on its own and a git failure aborts the commit.
- The staged-file list feeding the paren check had the same hole.
- New bats tests drive a broken git and pin both paths.
Diffstat (limited to 'modules/keybindings.el')
0 files changed, 0 insertions, 0 deletions