Home GnuPG
Diffusion Phabricator 545dad319ec0

Add an "Auditors" rule for Commits
545dad319ec0Unpublished
Actions

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Add an "Auditors" rule for Commits

Summary: Fixes T5889. You can't write a rule like "if no other Herald rules did anything...", but you can use this rule to check for Owners or an explicit "Auditors" field doing things.

Test Plan: Using the test console, ran an "Auditors" rule against a commit with and without an auditor. Got expected pass/fail outcomes.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T5889

Differential Revision: https://secure.phabricator.com/D17221

Details

Provenance
epriestley <git@epriestley.com>Authored on Jan 18 2017, 6:37 PM
Parents
rPHABa3b067291f85: Add a couple more files to the core JS package
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB545dad319ec0: Add an "Auditors" rule for Commits (authored by epriestley <git@epriestley.com>).Jan 18 2017, 7:05 PM

Changes (3)

PathSize
src/
applications/
diffusion/
herald/
repository/
storage/

rPHAB545dad319ec0

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/herald/DiffusionCommitAuditorsHeraldField.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryAuditRequest.php

Loading...