Page MenuHomePhabricator

Execute Herald again after promoting revisions out of the "Draft" state
ClosedPublic

Authored by epriestley on Dec 5 2017, 3:57 PM.
Tags
None
Referenced Files
F13237504: D18819.id.diff
Tue, May 21, 1:50 PM
Unknown Object (File)
Mon, Apr 29, 5:49 PM
Unknown Object (File)
Apr 19 2024, 7:01 PM
Unknown Object (File)
Apr 19 2024, 5:53 PM
Unknown Object (File)
Apr 16 2024, 9:35 PM
Unknown Object (File)
Apr 16 2024, 5:11 PM
Unknown Object (File)
Apr 13 2024, 5:57 AM
Unknown Object (File)
Apr 13 2024, 5:57 AM
Subscribers
None

Details

Summary

Fixes T13027. Ref T2543. When revisions promote from "Draft" because builds finish or no builds are configured, the status currently switches from "Draft" to "Needs Review" without re-running Herald.

This means that some rules -- notably, "Send me an email" rules -- don't fire as soon as they should.

Instead of applying this promotion in a hacky way inline, queue it and apply it normally in a second edit, after the current group finishes.

Test Plan
  • Created a revision, reviewed Herald transcripts.
  • Saw three Herald passes:
    • First pass (revision creation) triggered builds and no email.
    • Second pass (builds finished) did not trigger builds (no update) and did not trigger email (revision still a draft).
    • Third pass (after promotion out of 'draft') did not trigger builds (no update) but did trigger email (revision no longer a draft).

Diff Detail

Repository
rP Phabricator
Branch
draft1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18926
Build 25522: Run Core Tests
Build 25521: arc lint + arc unit