Infrastructure/fedora-infrastructure
1
0
Fork 0
Code Issues 70 Pull requests Projects Releases Packages Wiki Activity Actions

Remove bad_reviewers stuff

Browse source
This commit is contained in:
Jason Tibbitts 2012-06-13 21:07:51 -05:00
parent f5f479719d
commit 26f7b778a1
1 changed files with 0 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

6
scripts/process-git-requests/process-git-requests
View file

@ -51,9 +51,6 @@ epel_ok = ['abompard', 'athimm', 'corsepiu', 'ecik', 'faucamp', 'konradm',
epel_ok_comaint = ['alexlan', 'guidograzioli', 'jwrdegoede', 'kkofler', epel_ok_comaint = ['alexlan', 'guidograzioli', 'jwrdegoede', 'kkofler',
'mebourne', 'overholt', 'pgordon', 'rishi', 'snirkel'] 'mebourne', 'overholt', 'pgordon', 'rishi', 'snirkel']
# A hand-maintained list of reviewers who have done poor reviews in the past
bad_reviewers = ['rnovacek@redhat.com']
PAGER = os.environ.get('PAGER') or '/usr/bin/less' PAGER = os.environ.get('PAGER') or '/usr/bin/less'
EDITOR = os.environ.get('EDITOR') or '/bin/vi' EDITOR = os.environ.get('EDITOR') or '/bin/vi'
@ -480,9 +477,6 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc
if bug.assigned_to == 'nobody@fedoraproject.org': if bug.assigned_to == 'nobody@fedoraproject.org':
warnings.append("WARNING: Ticket is not assigned to anyone.") warnings.append("WARNING: Ticket is not assigned to anyone.")
warned = True warned = True
if bug.assigned_to in bad_reviewers:
warnings.append("WARNING: Reviewer %s has done poor reviews in the past; please check." % bug.assigned_to)
warned = True
# This is an unpleasant hack # This is an unpleasant hack
flags = bug.__getattr__('flagtypes.name') flags = bug.__getattr__('flagtypes.name')