On Sun, Sep 6, 2015 at 10:47 PM, Loic Dachary <loic@xxxxxxxxxxx> wrote: > Hi, > > Today I realized that when we ask for permission to merge [1], we don't usually assign the pull request to the person who is supposed to answer. This may not be of consequence if thre is just one pull request to look at. But it's not uncommon to have half a dozen such pull requests after a suite passed. When the person reads her/his mail, a few are addressed and more often than not the rest is postponed for when there is time. > > Only they don't show up anywhere in the person list of TODO things, therefore it's frequent that we have to send a reminder. Instead, it's probably easier on everyone if we assign the pull request to the person next to act in that specific case. It's not something I usually do for pull requests that are assigned to me, because it's the best way for me to completely forget about such pull requests. But in this case, since each pull request is linked to an issue and since we scrub these issues at least on a weekly basis, there is no risk of us forgetting. > > What do you think ? This is how I've been using PR assignment all along; sounds good to me! -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
