out of interest, I downloaded
http://trac.sagemath.org/query?status=closed&col=id&col=summary&col=status&col=milestone&col=resolution&col=changetime&col=commit&order=priority
as a .csv file and wrote an ugly script to see whether the commitid (if
present) is merged in 6.7.beta0. I then manually removed those tickets which
where closed yesterday, because that would be the tickets already closed for
6.7.beta1.
Here is the result
ticket commit changed
15017 872e2bc0225b6929ed8cc052a63850c892abe723 07/21/14 03:51:36
15660 d4842426356568815b80323f8fc6afbb80185ba3 01/25/15 14:55:21
15773 81a2ea983e2903b9cc5f2a78a57dd24fe32e48d5 08/31/14 19:45:31
16939 d9214c8c4ce4cbb5b29d207e8ed3bbf4e356b180 03/06/15 20:28:12
17221 2e62790eeec6463562120c204a86ebb323fa7cba 04/14/15 21:44:07
17307 7fa0de34aa713520d0fef2a7065581599dfd7bf3 11/09/14 20:40:11
14880 3b6a841dc5f6210a9c0d70b4d6cf24e34788c228 10/02/13 08:35:34
15599 08be4423f7703847d4da7c6d8cce4bb0902f93ed 03/13/14 04:38:46
16403 9f08c6b859a0fb853f69dbaedc73811740ee3e91 01/25/15 14:56:02
16847 e42ce551e66ff3f854c2424561bc71a075bac67a 08/19/14 12:38:59
16698 ba38202cf55caefb032e1b4519c96b9706577004 09/15/14 16:56:43
17702 daad37297eff79d7ee8c34989840d44f7e3e5105 02/08/15 17:31:25
#17221 (which started this thread) already has a successor ticket #18206; I did
not have a closer look at the others.
And for completeness, here is the ugly script (it assumes that develop is the
current branch):
import csv
import subprocess
with open('/tmp/query.csv', 'r') as csvfile:
tracreader = csv.reader(csvfile, delimiter=',', quotechar='"')
headings_row = next(tracreader)
headings_row[0] = 'id' # override UTF8 markers
for row in tracreader:
data = dict(zip(headings_row, row))
if data['commit'] and data['resolution'] == 'fixed':
try:
output = subprocess.check_output(["git", "branch",
"--contains", data['commit']],
stderr=subprocess.STDOUT).split('\n')
except subprocess.CalledProcessError:
print data['id'], data['commit'], data['changetime']
if "* develop" not in output:
print data['id'], data['commit'], data['changetime']
I am aware of the fact that this would be cleaner if implemented in git trac;
but for a first overview, this seemed to be the quicker approach.
Regards,
CH
Am 2015-04-16 um 05:50 schrieb Volker Braun:
> You are just renaming the step where nobody is supposed to make changes any
> more
> (from positive review to merging). It doesn't solve anything. Eventually a CI
> script will have to be written, and it'll automatically switch from positive
> review to merging.
>
> We could add a mandatory 2-week cooling off period while I'm not merging
> positively-reviewed tickets, if thats what you want. But I'd find that
> annoying.
--
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 http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.
