Arturo Borrero Gonzalez has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/391590 )
Change subject: commit_message_validator: homogeinize case in footers
......................................................................
commit_message_validator: homogeinize case in footers
Homogeinize case in commit message footers. The logic used is that
the '-by' part (i.e, the last part) is always lower case.
In the case of 'Change-Id', I left that untouched since may have impact
in how gerrit and other tools process changes.
While at it, add the missing 'Requested-by' line.
Change-Id: Iefc7f9811266cc629f5c43a008b612640a3d8030
Signed-off-by: Arturo Borrero Gonzalez <aborr...@wikimedia.org>
---
M commit_message_validator/__init__.py
1 file changed, 6 insertions(+), 5 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/integration/commit-message-validator
refs/changes/90/391590/1
diff --git a/commit_message_validator/__init__.py
b/commit_message_validator/__init__.py
index 09714e2..9715dea 100644
--- a/commit_message_validator/__init__.py
+++ b/commit_message_validator/__init__.py
@@ -40,16 +40,17 @@
# Header-like lines that we are interested in validating
CORRECT_FOOTERS = [
- 'Acked-By',
+ 'Acked-by',
'Bug',
'Cc',
'Change-Id',
- 'Co-Authored-By',
- 'Depends-On',
- 'Reported-By',
+ 'Co-Authored-by',
+ 'Depends-on',
+ 'Requested-by',
+ 'Reported-by',
'Reviewed-by',
'Signed-off-by',
- 'Suggested-By',
+ 'Suggested-by',
'Tested-by',
'Thanks',
]
--
To view, visit https://gerrit.wikimedia.org/r/391590
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iefc7f9811266cc629f5c43a008b612640a3d8030
Gerrit-PatchSet: 1
Gerrit-Project: integration/commit-message-validator
Gerrit-Branch: master
Gerrit-Owner: Arturo Borrero Gonzalez <aborr...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
