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.


Reply via email to