On Sun, Sep 2, 2012 at 5:19 PM, Jeroen Demeyer <jdeme...@cage.ugent.be> wrote: > On 2012-09-02 20:10, Simon King wrote: >> That's for the patchbot, which only reads the comments, but not the >> ticket description. > IMHO, that's still a bug in the patchbot. > >> If you want to help the release manager once the ticket has a positive >> review, also state the to-be-applied patches in the ticket description. > I would say: you *must* do this when it's not obvious which patches to > apply. You cannot rely on the fact that the release manager has read all > comments on the ticket. Besides, this also helps potential reviewers who > have not been following the whole history of the ticket.
Would it be possible to have a "patches to apply" field on the ticket (similar to the fields dependencies, keywords, etc.)? Then the release manager, reviewers and patchbots would easily know where to look. Take care, Franco -- -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To post to this group, send email to sage-devel@googlegroups.com. To unsubscribe from this group, send email to sage-devel+unsubscr...@googlegroups.com. Visit this group at http://groups.google.com/group/sage-devel?hl=en.