Allow Herald rules to check for revisions with no reviewers
ClosedPublic
Actions

Authored by epriestley on Dec 11 2013, 8:42 PM.

Tags

None

Referenced Files

	F19024231: D7757.diff
	Mon, Nov 24, 2:41 AM

	F19001013: D7757.id17544.diff
	Fri, Nov 21, 1:42 AM

	F18855456: D7757.id17544.diff
	Sat, Nov 1, 5:19 AM

	F18826420: D7757.id.diff
	Oct 24 2025, 3:55 AM

	F18825371: D7757.diff
	Oct 23 2025, 9:25 PM

	F18627531: D7757.diff
	Sep 16 2025, 4:15 AM

	F18624122: D7757.id17544.diff
	Sep 15 2025, 7:03 PM

	F18621691: D7757.id.diff
	Sep 15 2025, 9:08 AM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Maniphest Tasks

T4225: Allow Herald rules to apply the "exist" and "do not exist" conditions to the "Reviewers" field

Commits

Restricted Diffusion Commit
rP6b1ec35cf377: Allow Herald rules to check for revisions with no reviewers

Summary

Fixes T4225. Adds the NON_EXISTS condition to Herald for "Reviewers", and adds a few more conditions which have reasonable meanings.

Test Plan

Used test console to check a revision with reviewers, and another without reviewers. Both produced the expected results.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

coo

Closed by commit rP6b1ec35cf377 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

herald/

adapter/

HeraldAdapter.php

18 lines

storage/

HeraldRule.php

2 lines

Diff 17545

View Options

src/applications/herald/adapter/HeraldAdapter.php