Page MenuHomePhabricator
Differential D7820

Add a "branches" rule for Herald commit rules
ClosedPublic
Actions

Authored by epriestley on Dec 23 2013, 7:16 PM.
Tags
None
Referenced Files
F19072413: D7820.id.diff
Sun, Nov 30, 10:22 PM
F19063516: D7820.diff
Sat, Nov 29, 7:05 PM
F18851155: D7820.id.diff
Oct 30 2025, 10:36 PM
F18846481: D7820.diff
Oct 29 2025, 7:09 PM
F18759701: D7820.id17696.diff
Oct 6 2025, 5:27 AM
F18722241: D7820.id.diff
Sep 29 2025, 10:23 PM
F18630477: D7820.id17696.diff
Sep 16 2025, 11:45 AM
F18625262: D7820.id17707.diff
Sep 15 2025, 10:20 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4195: Implement pre-commit Herald rules
Commits
Restricted Diffusion Commit
rPadcc4ee1db1d: Add a "branches" rule for Herald commit rules
Summary

Fixes T4195. Allows you to write a rule against a commit's branches.

This completes outstanding work on T4195.

Test Plan

Pushed to Git and Mercurial repositories and verified branches were selected correctly by examining transcripts.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley added inline comments.Dec 23 2013, 7:17 PM
src/applications/diffusion/engine/DiffusionCommitHookEngine.php
431

This fixes a bug where we could update the wrong ref when a push includes multiple branches.

btrahan accepted this revision.Dec 26 2013, 6:22 PM
epriestley closed this revision.Dec 26 2013, 6:40 PM

Closed by commit rPadcc4ee1db1d (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
engine/
29 lines
herald/
7 lines
herald/
adapter/
3 lines

Diff 17707

View Options

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Loading...
View Options

src/applications/diffusion/herald/HeraldPreCommitContentAdapter.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
Phabricator · Built by Phacility