diff options
author | Clint Byrum <clint@fewbar.com> | 2018-04-06 07:05:07 -0700 |
---|---|---|
committer | Clint Byrum <clint@fewbar.com> | 2018-04-06 14:49:52 -0700 |
commit | 52aea0a417662b8a13566a23fc47a10e990e2b5e (patch) | |
tree | 778e369576a8da6aa194fa70ab9cff615d810442 /zuul/driver/github/githubsource.py | |
parent | c56edfbc74848eff2fb7716d8ebafdb8748961d4 (diff) | |
download | zuul-52aea0a417662b8a13566a23fc47a10e990e2b5e.tar.gz |
Make reject a mirror of require for github
The documentation claims that reject is a mirror of require, but only
reviews were included in the actual implementation. With these changes
we can now say the documentation is correct.
Change-Id: I3b321a38e0a2b36ec333ae99a97038cabbbc42d5
Diffstat (limited to 'zuul/driver/github/githubsource.py')
-rw-r--r-- | zuul/driver/github/githubsource.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/zuul/driver/github/githubsource.py b/zuul/driver/github/githubsource.py index 6f9b14d2c..6d9c29c53 100644 --- a/zuul/driver/github/githubsource.py +++ b/zuul/driver/github/githubsource.py @@ -140,7 +140,11 @@ class GithubSource(BaseSource): def getRejectFilters(self, config): f = GithubRefFilter( connection_name=self.connection.connection_name, - reject_reviews=to_list(config.get('review')) + reject_reviews=to_list(config.get('review')), + reject_labels=to_list(config.get('label')), + reject_statuses=to_list(config.get('status')), + reject_open=config.get('open'), + reject_current_patchset=config.get('current-patchset'), ) return [f] @@ -167,5 +171,9 @@ def getRequireSchema(): def getRejectSchema(): - reject = {'review': scalar_or_list(review)} + reject = {'status': scalar_or_list(str), + 'review': scalar_or_list(review), + 'open': bool, + 'current-patchset': bool, + 'label': scalar_or_list(str)} return reject |