qa/createWeeklyReport.py | 49 +++++++++++++++++++++++++++-------------------- 1 file changed, 29 insertions(+), 20 deletions(-)
New commits: commit 08175529f2ff18f1ce137b35460b65d541e8efe7 Author: Xisco Fauli <[email protected]> AuthorDate: Tue Nov 27 12:49:42 2018 +0100 Commit: Xisco Fauli <[email protected]> CommitDate: Tue Nov 27 12:50:47 2018 +0100 QA: Add top 10 commenters Suggested by Timur diff --git a/qa/createWeeklyReport.py b/qa/createWeeklyReport.py index 34d9432..a142e18 100755 --- a/qa/createWeeklyReport.py +++ b/qa/createWeeklyReport.py @@ -226,7 +226,10 @@ def analyze_bugzilla_weeklyReport(statList, bugzillaData, cfg): common.util_check_bugzilla_mail(statList, commentMail, '', commentDate, rowId) - if commentDate >= cfg['reportPeriod']: + if commentDate >= cfg['reportPeriod'] and \ + commentMail != "[email protected]" and\ + commentMail != "[email protected]" and\ + comment['text'] != 'A polite ping, still working on this bug?': if commentMail not in statList['comments_count']: statList['comments_count'][commentMail] = 0 statList['comments_count'][commentMail] += 1 @@ -298,7 +301,7 @@ def util_print_QA_line_weekly(fp, statList, dValue, action, isMetabug=False): else: print(text, file=fp) text = " " + personString + ", " - if text is not " ": + if text != " ": print(text[:-2], file=fp) print(file=fp) @@ -325,20 +328,29 @@ def create_weekly_Report(statList) : print(file=fp) d_view.sort(reverse=True) - print(' * Top 15 reporters:', file=fp) + print(' * Top 10 reporters:', file=fp) it = 0 for i1,i2 in d_view: - try: - if it >= 15: - break - print('\t\t+ ' + statList['people'][i2]['name'] + ' (' + str(i1) + ')', file=fp) - it += 1 - except: - continue + if it >= 10: + break + print('\t\t+ ' + statList['people'][i2]['name'] + ' (' + str(i1) + ')', file=fp) + it += 1 print(file=fp) + d_view = sorted(statList['comments_count'].items(), key=lambda kv: kv[1], reverse=True) + + print(' * Top 10 commenters:', file=fp) + + it = 0 + for i in d_view: + if it >= 10: + break + print('\t\t+ ' + statList['people'][i[0]]['name'] + ' (' + str(i[1]) + ')', file=fp) + it += 1 + + print(file=fp) print(" * {} bugs reported haven't been triaged yet.".format(\ len(statList['still_unconfirmed'])), file=fp) commit 6fa3ee954b092120c4cae4099f7a10565f34836b Author: Xisco Fauli <[email protected]> AuthorDate: Tue Nov 27 11:58:57 2018 +0100 Commit: Xisco Fauli <[email protected]> CommitDate: Tue Nov 27 12:50:47 2018 +0100 QA: Some names were left behind when breaking the lines diff --git a/qa/createWeeklyReport.py b/qa/createWeeklyReport.py index f09dc19..34d9432 100755 --- a/qa/createWeeklyReport.py +++ b/qa/createWeeklyReport.py @@ -291,16 +291,13 @@ def util_print_QA_line_weekly(fp, statList, dValue, action, isMetabug=False): text = " " for i1,i2 in d_view: - try: - personString = statList['people'][i2]['name'] + ' (' + str(i1) + ')' - # Reduce lines to 72 characters, for some reason the emails are cut otherwise - if len( text + " " + personString ) < 72: - text += personString + ", " - else: - print(text[:-2], file=fp) - text = " " - except: - continue + personString = statList['people'][i2]['name'] + ' (' + str(i1) + ')' + # Reduce lines to 72 characters, for some reason the emails are cut otherwise + if len( text + " " + personString ) < 72: + text += personString + ", " + else: + print(text, file=fp) + text = " " + personString + ", " if text is not " ": print(text[:-2], file=fp) _______________________________________________ Libreoffice-commits mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
