[llvm-dev] Phabricator default view
llvm-dev at lists.llvm.org
Mon Dec 3 11:05:26 PST 2018
I have to say, thanks for this. I discovered I'm a reviewer on a couple of things…
Is there a way to get "all open reviews that I'm subscribed to" (as opposed to, listed as a reviewer)?
From: llvm-dev [mailto:llvm-dev-bounces at lists.llvm.org] On Behalf Of Sam McCall via llvm-dev
Sent: Tuesday, November 27, 2018 1:11 PM
To: Michael Spencer
Cc: LLVM Dev
Subject: Re: [llvm-dev] Phabricator default view
Ah, I tried switching the default, and it doesn't work as I expected.
The "My activity" view isn't shown in the menu for logged-out users, but it's still selected as the homepage, so reviews.llvm.org<http://reviews.llvm.org> is an error page.
This seems like a blocker, I'll file a bug against phabricator.
On Tue, Nov 27, 2018 at 2:38 AM Michael Spencer <bigcheesegs at gmail.com<mailto:bigcheesegs at gmail.com>> wrote:
On Fri, Nov 23, 2018 at 5:59 AM Sam McCall via llvm-dev <llvm-dev at lists.llvm.org<mailto:llvm-dev at lists.llvm.org>> wrote:
Hi weary warriors of code reviewing,
The default homepage in phabricator leaves some things to be desired IMO:
- having changes sorted by *creation time* rather than *update time* is a fun way to lose track of things
- the LLVM-wide activity feed seems not that useful (though fun)
- as soon as a change lands, it becomes fairly hard to find
Fortunately phabricator is pretty customizable. I've added a couple of views to the main menu:
- My activity: https://reviews.llvm.org/home/menu/view/133/ is (active reviews, recent commits) x (i'm reviewer, i'm author). Only visible when logged in.
- All commits: https://reviews.llvm.org/home/menu/view/135/ is the firehose
I'd like to make "my activity" the default view for logged-in users, and "all commits" the default when logged out. (This is done by removing "home" from the menu).
These aren't optimal for any particular person (e.g. I'd like to see all clangd changes), and people should consider creating a custom dashboard if you use phabricator heavily. But I think they're reasonable defaults and a step up from what we have now.
Let me know what you think, especially if you prefer the status quo for some specific reason.
If people are generally positive, I'll change this next week. We can easily roll back.
I agree this is a much better default view for logged in users.
- Michael Spencer
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev