1483460 - trim bug number and reviewer from summary when generating bugzilla attachment description

Status: RESOLVED WONTFIX

(bugzilla.mozilla.org :: Phabricator Integration, enhancement)

Description

[Karl Tomlinson (:karlt)]

Not sure what is best for the review syntax.  It may be useful because review requests are not mapped to bugzilla requests, but it is only useful if accurate.

Bug 1483457 at least makes this misleading ATM.

Comment 1

[:glob ✱]

it isn't clear to me exactly what you're requesting.  can you provide examples of the behaviour you're seeing, and what you expect to see?

Comment 2

[Karl Tomlinson (:karlt)]

This is about generating a bugzilla attachment description from the first line
(summary) of a commit message.

I guess that has little to do with moz-phab, sorry.

For a summary
"bug 1483174 use float constants with float delay time variables r?padenot",
bzexport would describe the attachment as
"use float constants with float delay time variables"
That is the preferred description for the bugzilla attachment IMO.

Mozreview would describe the attachment as (and even change the description of
an existing unrelated attachment) to
"bug 1483174 use float constants with float delay time variables"

With moz-phab, I see
"Bug 1483174 use float constants with float delay time variables r=padenot"

I guess moz-phab changed the case and s/?/=/, but this bug is really about
removing "bug 1483174" and "r?padenot" entirely, which I guess is the
"Phabricator" component.  I'm not sure.

Comment 3

[Karl Tomlinson (:karlt)]

Having the Differential revision id somewhere in bugzilla would be useful too, to distinguish several attachments on the same bug (which sometimes all have the same commit message), and to determine which is associated with which differential revision without waiting for loads on each attachment.  The attachment description may be a good place if no mozreview-like UI will be added to bugzilla.

Comment 4

[:glob ✱]

since this bug was filed we've added a panel to bugzilla that shows the associated phabricator diffs.

as we plan on defocusing the bugzilla attachments we won't be making changes to improve their display.