Page MenuHomeSoftware Heritage
Herald H16

Non-developers require code review for pushing content
ActivePublic
Actions

Details

Rule Type
Global
Applies To
Commit Hook: Commit Content

Description

When all of these conditions are met:

Another Herald rule does not match: H31 Developers or CI
Accepted Differential revision does not exist
Repository projects include all of Developers

Take these actions every time this rule matches:

Block push with message: needs review (if you see this message despite having an accepted diff, make sure you ran "arc update" after your last change, including rebases)

Event Timeline

zack edited this rule. (Show Details)Apr 29 2016, 3:00 PM
zack renamed this rule from to Interns require code review for pushing.
zack edited this rule. (Show Details)Apr 29 2016, 3:26 PM
zack edited this rule. (Show Details)Apr 29 2016, 4:02 PM
zack edited this rule. (Show Details)May 3 2016, 5:08 PM
zack renamed this rule from Interns require code review for pushing to Interns require code review for pushing content.
zack edited this rule. (Show Details)
zack edited this rule. (Show Details)May 3 2016, 5:32 PM
zack disabled this rule.Jun 27 2016, 1:45 PM
zack enabled this rule.
zack edited this rule. (Show Details)Mar 28 2017, 11:00 AM
olasd edited this rule. (Show Details)May 5 2017, 4:49 PM
olasd edited this rule. (Show Details)Jun 14 2017, 3:28 PM
olasd edited this rule. (Show Details)Jun 29 2017, 7:53 PM
olasd edited this rule. (Show Details)Nov 27 2018, 12:52 AM
olasd edited this rule. (Show Details)Nov 27 2018, 1:09 AM
olasd renamed this rule from Interns require code review for pushing content to Non-developers require code review for pushing content.
vlorentz edited this rule. (Show Details)Dec 6 2021, 2:12 PM
About · Developer information · Legal