Jori Mäntysalo wrote: > On Fri, 1 Jul 2016, Erik Bray wrote: > >> One thing that will help, which has already been discussed up-thread, >> is having Trac help take care of the little nitty-gritty checks on >> tickets. > > True. It feels extremely stupid to get a message "You forgot reviewer's > name.", as this is exactly what computers should do instead of humans.
You cannot automatically add these names; all we could do is create some plug-in such that when a ticket is set to "positive review" (and perhaps also other states), the contents of the field gets checked. (Same for the author(s) when the ticket gets into "needs review".) If it's empty, we could offer a multiple-choice list of names of the participants of the ticket *mapped to their real names*, but our currently rather informal list of the latter [1] isn't complete. (A separate file / database would be good anyway, and trac already has something like that.) We may also change these fields to contain trac account names, but that's a matter of taste (and tradition), too. I think Jeroen *did* automate the posts regarding empty author and/or reviewer fields when he was release manager. :-) -leif [1] https://trac.sagemath.org/wiki#AccountNamesMappedtoRealNames -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at https://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/d/optout.
