esc-reporting/qa-tools.py | 26 +++++++++++++++++++++++--- 1 file changed, 23 insertions(+), 3 deletions(-)
New commits: commit 3ab9386571a956298adb720a3b56c0cf2070af5e Author: Xisco Fauli <[email protected]> Date: Thu Jan 25 13:22:33 2018 +0100 QA tools: Warn about new bugs moved to needinfo diff --git a/esc-reporting/qa-tools.py b/esc-reporting/qa-tools.py index 542dfa8..8596dc7 100755 --- a/esc-reporting/qa-tools.py +++ b/esc-reporting/qa-tools.py @@ -419,6 +419,8 @@ def analyze_bugzilla(statList, bugzillaData, cfg): reopener6MonthsEmail = "" isConfirmed = False movedToFixed = False + movedToNeedInfo = False + movedToNeedInfomail = "" isReopened = False reopenerEmail = "" @@ -507,6 +509,14 @@ def analyze_bugzilla(statList, bugzillaData, cfg): isReopened = True reopenerEmail = actionMail + if actionDate >= cfg['reportPeriod'] and addedStatus == 'NEEDINFO' and \ + rowStatus == 'NEEDINFO' and isOpen(removedStatus): + movedToNeedInfo = True + movedToNeedInfoMail = actionMail + + if movedToNeedInfo and removedStatus == 'NEEDINFO': + movedToNeedInfo = False + if actionDate >= cfg['reportPeriod'] and not bResolved and isClosed(addedStatus) and isClosed(row['status']): bResolved = True week = str(actionDate.year) + '-' + str(actionDate.strftime("%V")) @@ -543,7 +553,7 @@ def analyze_bugzilla(statList, bugzillaData, cfg): #if any other user moves it to open ( ASSIGNED, NEW or REOPENED ), #the bug is no longer autoconfirmed - if not everConfirmed and isOpen(rowStatus) and isOpen(addedStatus) and actionMail != creatorMail: + if not everConfirmed and isOpen(addedStatus) and actionMail != creatorMail: everConfirmed = True autoConfirmed = False @@ -846,6 +856,12 @@ def analyze_bugzilla(statList, bugzillaData, cfg): tup = (rowId, removeAssignedMail) lResults['removeAssigned'].append(tup) + if movedToNeedInfo and everConfirmed: + if 'movedToNeedInfo' not in lResults: + lResults['movedToNeedInfo'] = [] + tup = (rowId, movedToNeedInfoMail) + lResults['movedToNeedInfo'].append(tup) + if addAssignee: if 'addAssignee' not in lResults: lResults['addAssignee'] =[] commit f4a49dcecfaa043a98dfbe4dd3f4841c0ace87b7 Author: Xisco Fauli <[email protected]> Date: Wed Jan 24 18:36:41 2018 +0100 QA tools: check new contributors comments too diff --git a/esc-reporting/qa-tools.py b/esc-reporting/qa-tools.py index af18df4..542dfa8 100755 --- a/esc-reporting/qa-tools.py +++ b/esc-reporting/qa-tools.py @@ -693,7 +693,7 @@ def analyze_bugzilla(statList, bugzillaData, cfg): commentMail = comment['creator'] commentDate = datetime.datetime.strptime(comment['time'], "%Y-%m-%dT%H:%M:%SZ") - util_check_bugzilla_mail(statList, commentMail, '', commentDate) + util_check_bugzilla_mail(statList, commentMail, '', commentDate, rowId) util_increase_user_actions(statList, rowId, commentMail, bugTargets, 'comments', commentDate) if commentDate >= cfg['reportPeriod']: @@ -911,12 +911,16 @@ def analyze_bugzilla(statList, bugzillaData, cfg): if statList['people'][k]['oldest'] >= cfg['reportPeriod']: statList['weeklyReport']['newUsers'][k] = statList['people'][k] + if statList['people'][k]['oldest'] >= cfg['newUserPeriod'] and len(statList['people'][k]['bugs']) >= 3 and \ statList['people'][k]['email'] not in cfg['configQA']['ignore']['newContributors']: print('\n=== New contributor: '+ statList['people'][k]['name'] + " (" + statList['people'][k]['email'] + ")") lBugs = list(statList['people'][k]['bugs']) for idx in range(len(lBugs)): - print(str(idx + 1) + ' - ' + urlShowBug + str(lBugs[idx])) + isEasyHack = False + if 'easyHack' in bugzillaData['bugs'][str(lBugs[idx])]['keywords']: + isEasyHack = True + print(str(idx + 1) + ' - ' + urlShowBug + str(lBugs[idx]) + ' - easyHack: ' + str(isEasyHack)) statList['people'][k]['oldest'] = statList['people'][k]['oldest'].strftime("%Y-%m-%d") statList['people'][k]['newest'] = statList['people'][k]['newest'].strftime("%Y-%m-%d") _______________________________________________ Libreoffice-commits mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
