[llvm-dev] Upgrading phabricator
Aaron Ballman via llvm-dev
llvm-dev at lists.llvm.org
Fri Sep 30 06:26:41 PDT 2016
On Fri, Sep 30, 2016 at 9:21 AM, Eric Liu <ioeric at google.com> wrote:
> Thanks for the feedback Aaron! :)
> I've disabled it. I think the annoying part really is the status (e.g.
> Request, Closed etc) in the tag, and I am wondering if a tag with just line
> numbers like "(N Loc)" would be better. But I'm not really sure about the
> trade-off here.
Thank you for disabling it! Also, I agree, the Request, Closed, etc
stuff is also rather superfluous in some regards. But that at least
helps to determine whether the email is one I can archive immediately
or not (such as something that's been closed or abandoned), so perhaps
there is some utility with it? I do agree that it distracts from the
subject line, unfortunately.
> - Eric
> On Fri, Sep 30, 2016 at 3:05 PM Aaron Ballman <aaron at aaronballman.com>
>> On Fri, Sep 30, 2016 at 2:04 AM, Eric Liu via cfe-commits
>> <cfe-commits at lists.llvm.org> wrote:
>> > I've switched the default email format to be plain text only now. This
>> > option should be per-user configurable, but somehow it is not shown in
>> > the
>> > "Settings"; I'll try if I can make the option personalized.
>> Thank you for working on this upgrade!
>> Is there a way that we can remove the [Request, X lines] tag from the
>> email headers on phab patches? While it's neat to understand how large
>> a patch is, you can get a rough feel for that information by looking
>> at the patch contents. Unfortunately, the extra text means we lose
>> information from the subject line when viewing emails from a smaller
>> screen (like a mobile device), so I'm not certain that the utility
>> warrants the cost (at least, to me).
More information about the llvm-dev