Page MenuHomePhabricator

Prevent "Differential Revision: ..." from counting as a mention in commit messages
ClosedPublic

Authored by epriestley on May 22 2019, 9:15 PM.
Tags
None
Referenced Files
F13084006: D20544.diff
Wed, Apr 24, 10:46 PM
Unknown Object (File)
Tue, Apr 23, 6:42 AM
Unknown Object (File)
Sat, Apr 20, 4:49 PM
Unknown Object (File)
Thu, Apr 11, 7:40 AM
Unknown Object (File)
Thu, Apr 4, 3:33 AM
Unknown Object (File)
Mar 23 2024, 12:55 AM
Unknown Object (File)
Mar 23 2024, 12:55 AM
Unknown Object (File)
Mar 23 2024, 12:54 AM
Subscribers
None

Details

Summary

Ref T13290. Ref T13291. Now that a full URI is a "mention", the full URI in "Differential Revision: ..." also triggers a mention.

Stop it from doing that, since these mentions are silly/redundant/unintended.

The API here is also slightly odd; simplify it a little bit to get rid of doing "append" with "get + append + set".

Test Plan

Used bin/repository reparse --publish to republish commits with "Differential Revision: ..." and verified that the revision PHID was properly dropped from the mention list.

Diff Detail

Repository
rP Phabricator
Branch
ref1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22883
Build 31390: Run Core Tests
Build 31389: arc lint + arc unit