1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-15 11:22:40 +01:00
phorge-phorge/src/applications/herald
jungejason 4faab06c3c Enable herald rule for commits that need auditing
Summary:
enable herald commit rules to have access to auditing info.

Note that the new herald condition I added contains info for the
packages. I thought about using a simpler herald condition like
"Requires audit is true or false" and let it work together with the
existing "Affected package contains any of the package". It doesn't work
because we need the info about the package to decide if the commit
requires audit, but the herald conditions work separately.

Test Plan:
- A commit requiring auditing was detected by a herald rule that checks
  the auditing status
- A commit not requiring auditing was not detected by a herald rule
  which checks auditing status, but was detected by a rule which doesn't
  check the auditing status

Reviewers: epriestley, nh

Reviewed By: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1399
2012-01-17 09:13:07 -08:00
..
adapter Enable herald rule for commits that need auditing 2012-01-17 09:13:07 -08:00
config Enable herald rule for commits that need auditing 2012-01-17 09:13:07 -08:00
controller ...fix my fat finger period to a comma 2011-12-29 08:55:54 -08:00
engine Enable herald rule for commits that need auditing 2012-01-17 09:13:07 -08:00
storage Make Herald Rules sticky in X-Herald-Rules 2011-08-17 10:38:29 -07:00
view/rulelist Enable admin to view and delete other users' herald rules 2011-11-15 16:21:51 -08:00