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

Even more robust flag checking.

Browse source
This commit is contained in:
Jon Ciesla 2012-08-17 06:04:25 -05:00
parent fe057c77ae
commit e1b2f9aea0
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -485,16 +485,16 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc
warned = True
frflag = bug.get_flags('fedora-review')
if not frflag[0]['status']:
if not frflag:
warnings.append("WARNING: fedora-review flag not set")
warned = True
else:
if frflag[0]['status'] != '+':
warnings.append("WARNING: fedora-review flag not set to '+'")
warned = True
if frflag[0]['setter_email'] == bug.reporter:
warnings.append("WARNING: fedora-review flag set by review submitter! Verify that review was approved by reviewer!")
warned = True
if frflag[0]['setter_email'] == bug.reporter:
warnings.append("WARNING: fedora-review flag set by review submitter! Verify that review was approved by reviewer!")
warned = True
m=re.search('Review Request:\s+([a-zA-Z0-9_+.-]+)\s+', bug.summary, re.I)
if not m: