On Sun, Sep 6, 2015 at 10:47 PM, Loic Dachary <[email protected]> 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
