Page MenuHomePhabricator
Create Task
Maniphest T10649

more flexibility in Herald rules
Closed, ResolvedPublic
Actions

Description

I would like to track unreviewed commits against my master branch. An "unreviewed" commit is one that has neither a Phabricator Diff attached to it, nor the URL of our old code review tool in the commit message.

My rule currently looks like this

Screen Shot 2016-03-22 at 17.47.30.png (225×870 px, 27 KB)

The solution to my problem would be if I inverted all the rules - has a Diff, has text in commitmsg, is on master - and then change When to NONE of the rules. This only fixes my specific case and I imagine I could easily find one not as easily fixable.

Alternatively, a more complex UI with nested AND/OR blocks, would solve any case that can be described with the offered verbs.

Event Timeline

allixsenos created this task.Mar 22 2016, 4:50 PM
allixsenos added a comment.Mar 22 2016, 5:02 PM

... nvm. brain fart moment.

my rules work as expected. an audit was triggered by a someone submitting a diff that wasn't formally accepted.

epriestley added a subscriber: epriestley.Mar 22 2016, 5:02 PM

You can use two rules:

  • This is the first rule, with no actions ("Do Nothing").
  • The second rule uses an "[ Another Herald rule ][ does not match: ][ Your First Rule ]" condition to actually trigger actions.

Does that solve your issue?

allixsenos added a comment.Mar 22 2016, 5:03 PM

ah, chaining Herald rules, while clunky as a UI solution, probably fixes the more complex scenarios

epriestley closed this task as Resolved.Mar 22 2016, 5:05 PM
epriestley claimed this task.

Sounds like this is resolved one way or the other. In the general case, "Another Herald rule" effectively allows you to group, negate, and combine conditions in addition to reusing/including them.

(I think these kinds of very complex rules are unusual so I'd hesitate to add these capabilities as first-class in the UI without more/stronger use cases.)

allixsenos added a comment.Mar 22 2016, 5:06 PM

yeah, fair enough. if I build a stronger case I'll reopen/refile

Phabricator · Built by Phacility