Homec4science

Make execution order of Herald rules explicit

Authored by epriestley <git@epriestley.com> on Oct 2 2014, 18:49.

Description

Make execution order of Herald rules explicit

Summary: Fixes T6211. This gives Herald rules an explicit execution order, which seems generally good. See some discussion on T6211 and inline.

Test Plan:

  • Added unit test.
  • Dry ran rules and saw rules appear in the expected order in the transcript.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T6211

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

Details

Committed
epriestley <git@epriestley.com>Oct 2 2014, 18:49
Pushed
aubortJan 31 2017, 17:16
Parents
rPH3c6781b177d4: Differentiate between "no pygmetnize" and "nonworking pygmentize" during setup
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH5f8280506862: Make execution order of Herald rules explicit (authored by epriestley <git@epriestley.com>).Oct 2 2014, 18:49