On 17.03.25 23:11, Jelte Fennema-Nio wrote:
1. Major change: There's a new /me page which shows a dashboard of
open patches where you are author/reviewer/committer. These patches
are ordered & grouped in a hopefully useful way. Peter Geoghegan
suggested adding a "dashboard" of this kind. It's the first attempt
and can probably use some fine tuning based on feedback.

Thanks, this is very promising.

I have a few pieces of feedback that could be addressed by additional rules for how things are listed in the dashboard:

- If I'm the committer for a patch but not a reviewer, and the patch is in "needs review" status, then the patch is formally speaking not actionable by me and should not be under "Patches that are ready for your review". Perhaps it should be under "Blocked on others" [namely the reviewers], or in a different category.

- Conversely, if I'm the reviewer for a patch but not the committer, and the patch is in "ready for committer" status, then it's also not "Patches that are ready for your review". This might similarly be "Blocked on others" [namely the committer].

- Also, my dashboard shows patches from past and future commitfests. I don't know what I'm supposed to do with that. The purpose of having a "current" commitfest is that you work on that one when it's current. The patches in past and future commitfests were presumably put there for reasons that mean that they are not currently to be worked on as a priority.

(I'm probably not representative, but my current dashboard has a somewhat low hit ratio because of the above points.)



Reply via email to