Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33884 )
Change subject: util/lint: check for Change-Id line presence and duplicates ......................................................................
Patch Set 3: Code-Review+1
Patch Set 3:
Patch Set 3: -Code-Review
I double checked, and our lint scripts are executed in the pre-commit hook (but only the "stable" ones), which is run before the commit-msg hook that adds Change-Id.
So what's the intended behavior for this?
Meaning the file should be renamed (to lint-stable-024-change-id), in order to be run from jenkins?
No, jenkins runs both stable and extended (see util/testing/Makefile.inc:93) while pre-commit only runs stable (where this would break: Change-Id isn't added yet). So this is fine.
Please state the intent that this is for Jenkins' sanity tests in the commit message, because it appears to me that folks were wondering how this affects "git commit".