Current code checks spelling of names in commit message tags and that makes no sense.
Most of the tags are explicitly handled, but tags like 'Tested-by' or other lesser used ones are falling through to the spellchecker and need to be excluded. Signed-off-by: Ilya Maximets <[email protected]> --- utilities/checkpatch.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py index 692bb88d4..5a41bbef7 100755 --- a/utilities/checkpatch.py +++ b/utilities/checkpatch.py @@ -442,7 +442,8 @@ def check_spelling(line, comment): if not spell_check_dict or not spellcheck: return False - if line.startswith('Fixes: '): + is_name_tag = re.compile(r'^\s*([a-z-]+-by): (.*@.*)$', re.I | re.M | re.S) + if line.startswith('Fixes: ') or is_name_tag.match(line): return False words = filter_comments(line, True) if comment else line -- 2.44.0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
