Page MenuHomePhabricator

Modularize Herald Diffusion pre-commit content fields
ClosedPublic

Authored by epriestley on Jul 8 2015, 11:33 AM.

Details

Summary

Ref T8726. The gruntwork part of this is finally over.

Test Plan
  • Made a huge rule with every field.
  • Applied migration.
  • Verified the rule was still the same.
  • Pushed a bunch of commits and verified transcripts.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley retitled this revision from to Modularize Herald Diffusion pre-commit content fields.Jul 8 2015, 11:33 AM
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley updated this revision to Diff 32872.
btrahan edited edge metadata.Jul 8 2015, 6:59 PM
btrahan accepted this revision.
This revision is now accepted and ready to land.Jul 8 2015, 6:59 PM
This revision was automatically updated to reflect the committed changes.