Merge pull request #3151 from lkocman/acceptable_by_lawyer
Add state acceptable_by_lawyer
This commit is contained in:
commit
938421101f
@ -2,6 +2,9 @@
|
|||||||
|
|
||||||
# SPDX-License-Identifier: MIT
|
# SPDX-License-Identifier: MIT
|
||||||
|
|
||||||
|
# States from legaldb.suse.de / Cavil are documented in
|
||||||
|
# https://github.com/openSUSE/cavil/blob/master/docs/Architecture.md#states
|
||||||
|
|
||||||
import os
|
import os
|
||||||
import os.path
|
import os.path
|
||||||
import sys
|
import sys
|
||||||
@ -88,7 +91,7 @@ class LegalAuto(ReviewBot.ReviewBot):
|
|||||||
if not indexed:
|
if not indexed:
|
||||||
return False
|
return False
|
||||||
datetime = parse(indexed)
|
datetime = parse(indexed)
|
||||||
# give the legaldb 2 hours to find a match (so we prefer acceptable/correct over preliminary)
|
# give the legaldb 2 hours to find a match (so we prefer acceptable_by_lawyer/acceptable/correct over preliminary)
|
||||||
if datetime.now(timezone.utc) - datetime < timedelta(hours=2):
|
if datetime.now(timezone.utc) - datetime < timedelta(hours=2):
|
||||||
return False
|
return False
|
||||||
return True
|
return True
|
||||||
@ -136,7 +139,7 @@ class LegalAuto(ReviewBot.ReviewBot):
|
|||||||
if not self.dryrun:
|
if not self.dryrun:
|
||||||
REQ.patch(url, headers=self.legaldb_headers)
|
REQ.patch(url, headers=self.legaldb_headers)
|
||||||
continue
|
continue
|
||||||
if state not in ['acceptable', 'correct', 'unacceptable']:
|
if state not in ['acceptable_by_lawyer', 'acceptable', 'correct', 'unacceptable']:
|
||||||
return None
|
return None
|
||||||
if state == 'unacceptable':
|
if state == 'unacceptable':
|
||||||
user = report.get('reviewing_user', None)
|
user = report.get('reviewing_user', None)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user