Page MenuHomePhabricator
Differential D10316

Allow pre-commit adapter to use custom actions
ClosedPublic
Actions

Authored by hach-que on Aug 21 2014, 12:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 18, 7:53 AM
Unknown Object (File)
Fri, Jan 17, 11:15 AM
Unknown Object (File)
Thu, Jan 16, 11:51 AM
Unknown Object (File)
Wed, Jan 8, 2:46 PM
Unknown Object (File)
Fri, Jan 3, 8:26 AM
Unknown Object (File)
Sat, Dec 28, 8:29 AM
Unknown Object (File)
Sat, Dec 28, 8:29 AM
Unknown Object (File)
Sat, Dec 28, 8:29 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP2fd395e859bb: Allow pre-commit adapter to use custom actions
Summary

Looks like I missed this when implementing custom actions and hence you can't currently use custom actions on the pre-commit adapters.

Test Plan

Added a custom action to a pre-commit Herald rule.

Diff Detail

Repository
rP Phabricator
Branch
precommit-custom-action
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2298
Build 2302: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que updated this revision to Diff 24834.Aug 21 2014, 12:58 AM
hach-que retitled this revision from to Allow pre-commit adapter to use custom actions.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 21 2014, 12:58 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
hach-que planned changes to this revision.Aug 21 2014, 12:59 AM

Oops. Looks like that just fixes the UI; it doesn't fix the actual triggering of the actions.

Harbormaster completed remote builds in B2298: Diff 24834.Aug 21 2014, 12:59 AM
hach-que updated this revision to Diff 24835.Aug 21 2014, 1:02 AM
hach-que edited edge metadata.

Actually made it work :/

Harbormaster completed remote builds in B2299: Diff 24835.Aug 21 2014, 1:03 AM
epriestley accepted this revision.Aug 28 2014, 12:30 AM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/diffusion/herald/HeraldPreCommitAdapter.php
127

pht regression :o

This revision is now accepted and ready to land.Aug 28 2014, 12:30 AM
hach-que added inline comments.Aug 28 2014, 12:55 AM
src/applications/diffusion/herald/HeraldPreCommitAdapter.php
127

Ouch, these are missing pht in all the classes; I'll go fix the other adapters up as well.

hach-que updated this revision to Diff 24974.Aug 28 2014, 12:58 AM
hach-que edited edge metadata.

Fix pht() in other adapters as well

Harbormaster completed remote builds in B2379: Diff 24974.Aug 28 2014, 12:59 AM
hach-que closed this revision.Aug 28 2014, 12:59 AM
hach-que updated this revision to Diff 24975.

Closed by commit rP2fd395e859bb (authored by @hach-que).

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
herald/
22 lines

Diff 24834

View Options

src/applications/diffusion/herald/HeraldPreCommitAdapter.php

Loading...
Phabricator · Built by Phacility