HomePhabricator

gerrit: Don't match Phabricator identifiers within urls

Description

gerrit: Don't match Phabricator identifiers within urls

Split the 'its-phabricator' rule into two:

  • 'its-phabricator' is the magic for Phabricator notification bot, and now just doesn't convert the matches text. The limitations of Gerrit's its-system make it impossible to do well (it requires the issue-number to be in the first matching group).
  • 'phabricator' is where linking actually happens, mimicked on the existing 'commit' rule. As far as I know this has no magic connected.

Samples:

Bug: T75997
Change-Id: Ib66d7d16cce54cc7962db2f15770f0b4493907ef