Page MenuHomePhabricator

Fix a bad interaction between "arc diff --reviewers" and "the first line of a message is always a title"
ClosedPublic

Authored by epriestley on Jan 5 2017, 10:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 14, 9:43 AM
Unknown Object (File)
Sat, Jul 13, 1:00 PM
Unknown Object (File)
Thu, Jul 11, 9:49 PM
Unknown Object (File)
Tue, Jul 9, 11:44 AM
Unknown Object (File)
Sat, Jul 6, 10:56 AM
Unknown Object (File)
Wed, Jul 3, 5:04 PM
Unknown Object (File)
Wed, Jul 3, 4:16 PM
Unknown Object (File)
Sun, Jun 30, 1:30 PM
Subscribers
None

Details

Summary

Fixes T12069. We implement "arc diff --reviewers" (and "--cc") by parsing a faux message with "Reviewers: ...".

After D17122, the first line of the message is always interpreted as a title, so the text ends up in the message body.

Instead, use a placeholder title so these fields are never initial fields.

Test Plan

Ran arc diff --reviewers dog, got only one "Reviewers" field.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable