CodeReviewBot is intended for use with GitLab-Phabricator integrations.
Has a generic name in case we want to use it with tools other than GitLab
at any point.
For https://phabricator.wikimedia.org/T330923
Bug: T330923
# Subject: summary of your change
# * "If applied, this commit will..."
# * Imperative mood: Change/Add/Fix/Remove/Update/Refactor/Document
# * Optionally, prefix with subject / component (general area modified)
# * https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines#Subject
# Body:
# * Why? What's wrong with status quo? Are there other ways?
# * How to test/confirm?
# * https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines#Body
# Bug: TXXXXXX
#
# Other fields:
# ~/code/bpb-kit/home/cheatsheets/git-meta.txt
Change the gerrit_patch_id regexp from
r'https://gerrit(?:-test|)\.wikimedia\.org/r/.*(\d+)(?:$|\]\])'
to
r'https://gerrit(?:-test|)\.wikimedia\.org/r/(\d+)(?:$|\]\])'
The .* in the old regexp was collecting most of the digits expected to
be matched by the (\d+). For example if the input was
https://gerrit.wikimedia.org/r/875932, the collected digits would just
be "2".