backport_pr: exclude 'Reviewed' labels from backport

This commit is contained in:
Martine Lenders 2019-04-24 11:47:49 +02:00
parent 292c5dd730
commit e3a99c40f6

View File

@ -26,7 +26,7 @@ WORKTREE_SUBDIR = "backport_temp"
RELEASE_PREFIX = "" RELEASE_PREFIX = ""
RELEASE_SUFFIX = "-branch" RELEASE_SUFFIX = "-branch"
LABELS_REMOVE = ['Process: needs backport'] LABELS_REMOVE = ['Process: needs backport', 'Reviewed: ']
LABELS_ADD = ['Process: release backport'] LABELS_ADD = ['Process: release backport']
BACKPORT_BRANCH = 'backport/{release}/{origbranch}' BACKPORT_BRANCH = 'backport/{release}/{origbranch}'
@ -36,14 +36,16 @@ def _get_labels(pr):
""" """
>>> _get_labels({'labels': [{'name': 'test'}, {'name': 'abcd'}]}) >>> _get_labels({'labels': [{'name': 'test'}, {'name': 'abcd'}]})
['Process: release backport', 'abcd', 'test'] ['Process: release backport', 'abcd', 'test']
>>> _get_labels({'labels': [{'name': 'Reviewed: what'}, {'name': 'Reviewed: 3-testing'}]})
['Process: release backport']
>>> _get_labels({'labels': [{'name': 'Process: release backport'}]}) >>> _get_labels({'labels': [{'name': 'Process: release backport'}]})
['Process: release backport'] ['Process: release backport']
>>> _get_labels({'labels': [{'name': 'Process: needs backport'}]}) >>> _get_labels({'labels': [{'name': 'Process: needs backport'}]})
['Process: release backport'] ['Process: release backport']
""" """
labels = {label['name'] for label in pr['labels']} labels = set(label['name'] for label in pr['labels']
for remove in LABELS_REMOVE: if all(not label['name'].startswith(remove)
labels.discard(remove) for remove in LABELS_REMOVE))
labels.update(LABELS_ADD) labels.update(LABELS_ADD)
return sorted(list(labels)) return sorted(list(labels))