Add license/consensus specific options

Author: tedivm

README.md

@@ -49,6 +49,18 @@ delayoverride: 10
 # When `delayoverride` is set this value is the minimum hours without changes before the PR will be merged
 mergedelaymin: 1
 
+# Require this amount of time in hours before a PR with a license change will be merged.
+licensedelay: 72
+
+# Require this amount of time in hours before a PR with a consensus change will be merged.
+consensusdelay: 72
+
+# Do not allow license changes to be merged.
+licenselock: true
+
+# Do not allow consensus changes to be merged.
+consensuslock: true
+
 # Number of hours after last action (commit or opening the pull request) before issue is autoclosed
 timeout: 720
 ```

gitconsensus/gitconsensus.py

@@ -51,6 +51,7 @@ def info(username, repository_name, pull_request):
     consensus = repo.getConsensus()
     click.echo("Mergeable:    %s" % (consensus.isMergeable(request),))
     click.echo("Is Blocked:   %s" % (request.isBlocked(),))
+    click.echo("Is Allowed:   %s" % (consensus.isAllowed(request),))
     click.echo("Has Quorum:   %s" % (consensus.hasQuorum(request),))
     click.echo("Has Votes:    %s" % (consensus.hasVotes(request),))
     click.echo("Has Aged:     %s" % (consensus.hasAged(request),))

gitconsensus/repository.py

@@ -369,6 +369,8 @@ def __init__(self, rules):
     def validate(self, pr):
         if pr.isBlocked():
             return False
+        if not self.isAllowed(pr):
+            return False
         if not self.isMergeable(pr):
             return False
         if not self.hasQuorum(pr):
@@ -379,6 +381,15 @@ def validate(self, pr):
             return False
         return True
 
+    def isAllowed(self, pr):
+        if pr.changesLicense():
+            if 'locklicense' in self.rules and self.rules['locklicense']:
+                return False
+        if pr.changesConsensus():
+            if 'lockconsensus' in self.rules and self.rules['lockconsensus']:
+                return False
+        return True
+
     def isMergeable(self, pr):
         if not pr.pr.mergeable:
             return False
@@ -401,9 +412,17 @@ def hasVotes(self, pr):
         return True
 
     def hasAged(self, pr):
+        hours = pr.hoursSinceLastUpdate()
+        if pr.changesLicense():
+            if 'licensedelay' in self.rules and self.rules['licensedelay']:
+                if hours < self.rules['licensedelay']:
+                    return False
+        if pr.changesConsensus():
+            if 'consensusdelay' in self.rules and self.rules['consensusdelay']:
+                if hours < self.rules['consensusdelay']:
+                    return False
         if 'mergedelay' not in self.rules:
             return True
-        hours = pr.hoursSinceLastUpdate()
         if hours >= self.rules['mergedelay']:
             return True
         if 'delayoverride' in self.rules and self.rules['delayoverride']: