Page MenuHomePhabricator

Use DifferentialCommitMessageParser and Modular Transactions to implement "Auditors: ..."
ClosedPublic

Authored by epriestley on Jan 30 2017, 5:53 PM.
Tags
None
Referenced Files
F15419978: D17262.id41528.diff
Fri, Mar 21, 10:19 AM
F15417865: D17262.id41538.diff
Thu, Mar 20, 7:04 PM
F15414256: D17262.id.diff
Wed, Mar 19, 11:43 PM
F15408201: D17262.diff
Tue, Mar 18, 9:39 PM
F15397777: D17262.id41528.diff
Sun, Mar 16, 10:14 PM
F15385690: D17262.id41538.diff
Fri, Mar 14, 11:11 PM
F15384760: D17262.diff
Fri, Mar 14, 8:44 PM
F15382070: D17262.id41528.diff
Fri, Mar 14, 10:32 AM
Subscribers
None

Details

Summary

Ref T10978. Updates how we implement "Auditors: ..." in commit messages:

  • Use the same parsing code as everything else.
    • (Also: parse package names.)
  • Use the new transaction code.

Also, fix some UI strings.

Test Plan

Used bin/repository reparse --herald <commit> to re-run this code on commits with various messages (valid Auditors, invalid Auditors, no Auditors). Saw appropriate auditors added in the UI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable