Page MenuHomePhabricator
Differential D7789

Provide basic commit content hooks for Herald
ClosedPublic
Actions

Authored by epriestley on Dec 18 2013, 8:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 6:10 PM
Unknown Object (File)
Thu, Apr 11, 10:00 AM
Unknown Object (File)
Sat, Apr 6, 6:35 AM
Unknown Object (File)
Tue, Apr 2, 8:47 PM
Unknown Object (File)
Mon, Apr 1, 1:41 PM
Unknown Object (File)
Mon, Apr 1, 1:26 PM
Unknown Object (File)
Mon, Apr 1, 1:26 PM
Unknown Object (File)
Mon, Apr 1, 12:58 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4195: Implement pre-commit Herald rules
Commits
Restricted Diffusion Commit
rPe115f11f8049: Provide basic commit content hooks for Herald
Summary

Ref T4195. This doesn't provide any interesting fields yet (content, affected paths, commit message) but fires the hook correctly.

Test Plan

Added a blocking hook and saw it fire.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
diffusion/
engine/
45 lines
herald/
120 lines
herald/
storage/
2 lines

Diff 17627

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Loading...
View Options

src/applications/diffusion/herald/HeraldPreCommitContentAdapter.php

Loading...
View Options

src/applications/herald/storage/HeraldRule.php

Loading...
Phabricator · Built by Phacility