Page MenuHomePhabricator

Restructure HookEngine to use PushLog records for all operations
ClosedPublic

Authored by epriestley on Dec 12 2013, 8:24 PM.
Tags
None
Referenced Files
F19553578: D7761.diff
Thu, Jan 29, 12:25 PM
F19511928: D7761.id17556.diff
Mon, Jan 12, 9:37 AM
F19154503: D7761.id.diff
Dec 11 2025, 3:46 PM
F19080425: D7761.id17556.diff
Dec 2 2025, 1:28 AM
F19076366: D7761.id17596.diff
Dec 1 2025, 12:09 PM
F19038643: D7761.id17596.diff
Nov 26 2025, 4:42 AM
F19038252: D7761.id17556.diff
Nov 26 2025, 3:45 AM
F18966374: D7761.diff
Nov 14 2025, 7:05 PM
Subscribers

Details

Summary

Ref T4195. This pulls the central logic of HookEngine up one level and makes all the git stuff genrate PushLogs.

In future diffs, everything will generate PushLogs and we can hand those off to Herald.

Test Plan

Pushed a pile of valid/invalid stuff:

{F89256}

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan added inline comments.
src/applications/diffusion/engine/DiffusionCommitHookEngine.php
445–447

smart