Page MenuHomePhabricator

Modularize all Herald pre-commit ref fields
ClosedPublic

Authored by epriestley on Jul 8 2015, 10:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 18, 6:37 AM
Unknown Object (File)
Tue, Apr 16, 4:24 AM
Unknown Object (File)
Tue, Apr 16, 3:06 AM
Unknown Object (File)
Wed, Apr 10, 7:39 PM
Unknown Object (File)
Tue, Apr 9, 12:32 PM
Unknown Object (File)
Tue, Apr 9, 5:50 AM
Unknown Object (File)
Sat, Apr 6, 3:06 PM
Unknown Object (File)
Sat, Apr 6, 1:27 PM
Subscribers

Details

Summary

Ref T8726. Just (pre-commit content) one more left.

Test Plan
  • Created a big rule with every field.
  • Migrated it.
  • Verified the rule was still the same.
  • Pushed a bunch of changes and reviewed the transcripts.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Modularize all Herald pre-commit ref fields.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
  • Also handle special rendering of "Change" rule.
btrahan edited edge metadata.
btrahan added inline comments.
src/applications/herald/adapter/HeraldAdapter.php
1215–1223

nice

This revision is now accepted and ready to land.Jul 8 2015, 7:00 PM
This revision was automatically updated to reflect the committed changes.