Last month I cleaned the git-history incorrect email addresses: https://edk2.groups.io/g/devel/message/51834 but today I noticed more incorrect addresses got committed (see caa917491a4..33a3293651). To avoid having the same cleanup in the future, fix the problem once for all by having PatchCheck doing this automatically (on merge).
Since we already have code to check email address in PatchCheck, factor the code out to reuse it, and add checks for committer/author addresses. Series only tested on Linux. Since v3 [1]: - Correct maintainer Cc tag in patch description Since v2 [2]: - Added 2020 Copyright notice in header - Added Bob Feng Reviewed-by tag Since v1 [3]: - Addressed Jordan Justen review comment [1] https://edk2.groups.io/g/devel/message/53062 [2] https://edk2.groups.io/g/devel/message/52656 [3] https://edk2.groups.io/g/devel/message/52885 Cc: Bob Feng <bob.c.f...@intel.com> Cc: Liming Gao <liming....@intel.com> Philippe Mathieu-Daudé (4): BaseTools/PatchCheck.py: Extract email check code to EmailAddressCheck BaseTools/PatchCheck.py: Let EmailAddressCheck describe email checked BaseTools/PatchCheck.py: Check the patch author email address BaseTools/PatchCheck.py: Check the committer email address BaseTools/Scripts/PatchCheck.py | 100 ++++++++++++++++++++++---------- 1 file changed, 70 insertions(+), 30 deletions(-) -- 2.21.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#53067): https://edk2.groups.io/g/devel/message/53067 Mute This Topic: https://groups.io/mt/69576944/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
