Robert Bradshaw <rober...@math.washington.edu> writes: > On Fri, Mar 2, 2012 at 1:45 AM, Keshav Kini <keshav.k...@gmail.com> wrote: >> For example, if two >> people have branches named trac-n, the script should just do nothing, >> not try to pick the most recently updated one, or look for branch names >> in the comments, or whatever :) > > Or one could preemptively test both, assuming the cycles are > available. For someone at the command line trying to review/pull a > ticket, it should ask.
I wasn't talking about the patchbot there, actually, just a trac extension to grok what was "going on" in people's code wrt a ticket and display something about it on the trac ticket page. I would actually like the patchbot to NOT do continuous builds and tests, once we move to a push/pull system. There should be a big button on each ticket which says "Test Me!", which will cause the patchbot to fetch whatever branch is specified on the ticket (in some "branch" field) and put the pair "(ticket, commit-id)" into a queue. (The patchbot wouldn't actually test commit-id, but would attempt to temporarily merge commit-id into master, and then test the resulting commit or return a failure to automatically merge / report that the branch needed merging / rebasing.) One reason for this: once we switch to a push/pull system, commits will hopefully be a lot more frequent, and I question whether we would even want the patchbot to test each and every commit over the whole doctest suite. Of course, it would be fine for the patchbot to at least notice that its last test was out of date and mention this on the trac ticket somewhere - maybe we could have a little "patchbot info area" <div> instead of the current patchbot icon. What do you think? -Keshav ---- Join us in #sagemath on irc.freenode.net ! -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
