Hmm, no, I'm confused. That message was from the editor. The hooks do get run, but they let it through.
I was expecting the foreign_ident check to trig there. It misses the file if I do --amend, but not if I reset and commit:
git commit --amend -C HEAD
[detached HEAD c531787] Avoid yielding to other threads in some places where the lvalue is temporarily cleared for single ref optimizations. 1 files changed, 16 insertions(+), 2 deletions(-)
git reset HEAD^
Unstaged changes after reset: M src/interpret_functions.h
git commit -a -C c531787
File src/interpret_functions.h has the foreign_ident attribute. Please remove it before commit.