HomePhorge

Diffusion / Herald - tweak "Accepted Differential Revision exists" Herald field

Description

Diffusion / Herald - tweak "Accepted Differential Revision exists" Herald field

Summary: make it use the value of the revision before any post-commit magic has occurred. Fixes T4754

Test Plan: made a herald rule that said "if revision exists, and revision accept does not exists, block push". tried to push a commit that had a revision that wasn't accepted and I was blocked.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: mbishopim3, epriestley, Korvin

Maniphest Tasks: T4754, T4574

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

Details

Provenance
Bob TrahanAuthored on Sep 3 2014, 3:28 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP7ecbc707846f: Daemons - handle daemons that can't be killed a bit better
Branches
Unknown
Tags
Unknown

Event Timeline