[Libreoffice-commits] dev-tools.git: 2 commits - esc-reporting/qa-tools.py

[Xisco Fauli]

 esc-reporting/qa-tools.py |   26 +++++++++++++++++++++++---
 1 file changed, 23 insertions(+), 3 deletions(-)

New commits:
commit 3ab9386571a956298adb720a3b56c0cf2070af5e
Author: Xisco Fauli <anistenis at gmail.com>
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 <anistenis at gmail.com>
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")