HomePhorge

Enable herald rule for commits that need auditing

Description

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

Details

Provenance
jungejasonAuthored on Jan 13 2012, 11:08 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP05ee317555a0: Remove "parsedHunk" property
Branches
Unknown
Tags
Unknown

Event Timeline